Hi, it seems moderation didn't approve the phabricator mails for D12834. (I have no intention to be subscribed to the list just to get phabricator mails through. For now I am subscribed but disabled mail delivery -.-)
So the patch is available at http://reviews.llvm.org/D12834 I also added a test case which shows that substitution is not working correctly right now. regards, Stefan PS: I think contributing to llvm is way to complex. First I attatch patches in the bug tracker, and it takes ages to get a reaction - and them I'm simply told to send to the ML. Then you want me to use phabricator, and then I get no feedback again - because moderation blocked emails from an internal system, and it seems nobody is looking at phabricator directly. This is not something I look forward to go through again; my motiviation to contribute further is rather low. On Sat, 12 Sep 2015 12:36:19 -0700 David Majnemer <david.majne...@gmail.com> wrote: > Would you mind sticking abi-tag.patch in phabricator: > http://llvm.org/docs/Phabricator.html ? > Your patch is big enough that it would make it a little easier for me > to review. > > On Sat, Sep 12, 2015 at 7:12 AM, Stefan Bühler via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > > > Hi all, > > > > I've been working on #23529. > > > > The abi tag mangling implemented by gcc is horrible, but I think my > > patch covers most of the incompatibilities with gcc5. > > > > There might be some bugs with substitutions, although I have to > > come up with a test case for that to see what gcc does... > > > > Test cases comparing gcc and patched clang++: > > http://files.stbuehler.de/test-itanium-mangle.html > > (generated with http://files.stbuehler.de/test-itanium-mangle.sh, > > also attached) _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
