[cfe-commits] [PATCH] Mangler does not mangle extern "C"'d symbols in ::std correctly

John McCall rjmccall at apple.com
Wed Mar 7 21:53:04 PST 2012

On Mar 7, 2012, at 9:41 PM, John McCall wrote:
> On Mar 2, 2012, at 7:08 AM, James Molloy wrote:
>> The attached patch fixes a bug in which a name in namespace ::std which is
>> within an extern "C" {} block* would not be mangled correctly. The patch
>> itself is simple, and comes from the lack of a call to
>> IgnoreLinkageSpecDecls().
>> As part of general code smell I have also moved two other places where
>> IgnoreLinkageSpecDecls()'s functionality was being duplicated to using that
>> function.
>> Regression test attached. Is it OK to commit?
> Looks good.

I see that my mail client is just woefully mangling the threading and this is long-ago reviewed.  Carry on.


More information about the cfe-commits mailing list