<div dir="ltr"><div><div><div><div>Hi Bert,<br><br></div>I'm not really sure why this patch was committed with this commit message, I'm sorry for the confusion.  The real patch which the commit message describes was committed in r<span class="">249437 (and reviewed in <a href="http://reviews.llvm.org/D13317">http://reviews.llvm.org/D13317</a>.)<br><br></span></div><span class="">I'll revert this and re-commit it with a better commit message.<br><br></span></div><span class="">Cheers,<br></span></div><span class="">Ehsan<br></span></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 30, 2016 at 1:45 AM, Bert Wesarg <span dir="ltr"><<a href="mailto:bert.wesarg@googlemail.com" target="_blank">bert.wesarg@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Dear Ehsan,<br>
<br>
the commit message from r253909 reads:<br>
<br>
Make clang_Cursor_getMangling not mangle if the declaration isn't mangled<br>
<br>
Right now clang_Cursor_getMangling will attempt to mangle any<br>
declaration, even if the declaration isn't mangled (extern C).  This<br>
results in a partially mangled name which isn't useful for much. This<br>
patch makes clang_Cursor_getMangling return an empty string if the<br>
declaration isn't mangled.<br>
<br>
But the code looks like it is handling only the visibility attribute<br>
of symbols. Where is the real r253909?<br>
<br>
Thanks.<br>
<br>
Best,<br>
Bert<br>
<br>
</div></div>PS: The mail from the original author does not work anymore: Michael<br>
Wu <<a href="mailto:mwu@mozilla.com">mwu@mozilla.com</a>><br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div>