Bug 14178 – C++ linux name mangling does not handle standard abbreviations for const types

There is a name mangling issue in dmd related to the compression of usual stl types when the type is const eg. dmd will mangle 'std::vector<int>::size() const' as '_ZNK3std6vectorIiSaIiEE4sizeEv' where it should be mangled '_ZNKSt6vectorIiSaIiEE4sizeEv'. Note 'std' is being compressed into 'St'. I believe this is coming from https://github.com/D-Programming-Language/dmd/blob/master/src/cppmangle.c#L453 which tests only the non const type pattern. According to http://mentorembedded.github.io/cxx-abi/abi.html#mangling-compression this substitution seems illegal "Note that substitutable components are the represented symbolic constructs, not their associated mangling character strings." This lead to undefined reference when linking.

FYI, I'm working on this one right now.

I think we aren't handling the optional CV-qualifier [0] correctly. [0] http://mentorembedded.github.io/cxx-abi/abi.html#mangle.type

Many things are broken with the current implementation : template substitutions, abbreviations. I'm rewriting the mangling code but it's quite complicated. The test suite is pretty incomplete too, this is the first thing I want to fix.

I have a hacky implementation for that particular issue, but if you rewritting the mangeling code it might hold off.

I expect it to take quite some time for me to have it right. This might be worth submitting you fix meanwhile.

(In reply to Guillaume Chatelet from comment #3) > Many things are broken with the current implementation : template > substitutions, abbreviations. It'd be nice to see a list of these.

Walter, here is my test suite https://github.com/gchatelet/dmd/tree/new_cpp_mangling2/test/mangling

any progress with this?

Unfortunately no. I can't find the time to work on this anymore :(

*** Issue 18957 has been marked as a duplicate of this issue. ***

Commit pushed to master at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/cce909b19a1e6b03a182d2a1d488841e2cc64b53 Merge pull request #8341 from TurkeyMan/mangle_std Fix issue 14178 - extern(C++) doesn't mangle 'std' correctly on posix systems merged-on-behalf-of: Mathias LANG <[email protected]>