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

Eli Friedman eli.friedman at gmail.com
Fri Mar 2 12:00:24 PST 2012


On Fri, Mar 2, 2012 at 7:08 AM, James Molloy <james.molloy at arm.com> wrote:
> Hi,
>
> 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?

I don't see the regression test... but otherwise, patch looks fine.

-Eli



More information about the cfe-commits mailing list