<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Sun, Sep 27, 2015 at 2:45 PM Stefan Bühler via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
it seems moderation didn't approve the phabricator mails for D12834.<br>
(I have no intention to be subscribed to the list just to get<br>
phabricator mails through. For now I am subscribed but disabled<br>
mail delivery -.-)<br>
<br>
So the patch is available at <a href="http://reviews.llvm.org/D12834" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12834</a><br>
<br>
I also added a test case which shows that substitution is not<br>
working correctly right now.<br>
<br>
regards,<br>
Stefan<br>
<br>
PS: I think contributing to llvm is way to complex. First I attatch<br>
patches in the bug tracker, and it takes ages to get a reaction - and<br>
them I'm simply told to send to the ML. Then you want me to use<br>
phabricator, and then I get no feedback again - because moderation<br>
blocked emails from an internal system, and it seems nobody is looking<br>
at phabricator directly.<br>
This is not something I look forward to go through again; my<br>
motiviation to contribute further is rather low.<br></blockquote><div><br></div><div>I'm very sorry you had a bad experience.</div><div><br></div><div>That patches attached to bugs don't get answered is an unfortunate issue; the problem is that most patches are when sent out first not of a quality that they can be directly submitted, and require significant work (this is especially true for larger packages), which is why we have a code review process set up via the list. Could we perhaps help that with better documentation somewhere? Where would you have looked first?</div><div><br></div><div>That the phab email was blocked without getting moderator approval is surprising to me. I'm going to figure out who moderates the email list these days, and why that didn't get unstuck.</div><div>Also, note that phab is generally not required (but it will often get you code reviews faster, as especially large patches are harder to review for some people).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sat, 12 Sep 2015 12:36:19 -0700<br>
David Majnemer <<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>> wrote:<br>
<br>
> Would you mind sticking abi-tag.patch in phabricator:<br>
> <a href="http://llvm.org/docs/Phabricator.html" rel="noreferrer" target="_blank">http://llvm.org/docs/Phabricator.html</a> ?<br>
> Your patch is big enough that it would make it a little easier for me<br>
> to review.<br>
><br>
> On Sat, Sep 12, 2015 at 7:12 AM, Stefan Bühler via cfe-commits <<br>
> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
><br>
> > Hi all,<br>
> ><br>
> > I've been working on #23529.<br>
> ><br>
> > The abi tag mangling implemented by gcc is horrible, but I think my<br>
> > patch covers most of the incompatibilities with gcc5.<br>
> ><br>
> > There might be some bugs with substitutions, although I have to<br>
> > come up with a test case for that to see what gcc does...<br>
> ><br>
> > Test cases comparing gcc and patched clang++:<br>
> > <a href="http://files.stbuehler.de/test-itanium-mangle.html" rel="noreferrer" target="_blank">http://files.stbuehler.de/test-itanium-mangle.html</a><br>
> > (generated with <a href="http://files.stbuehler.de/test-itanium-mangle.sh" rel="noreferrer" target="_blank">http://files.stbuehler.de/test-itanium-mangle.sh</a>,<br>
> > also attached)<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>