diff options
| -rw-r--r-- | clang/lib/CodeGen/Mangle.cpp | 10 | ||||
| -rw-r--r-- | clang/test/CodeGen/mangle.c | 9 |
2 files changed, 14 insertions, 5 deletions
diff --git a/clang/lib/CodeGen/Mangle.cpp b/clang/lib/CodeGen/Mangle.cpp index f007539714c..f310e989b51 100644 --- a/clang/lib/CodeGen/Mangle.cpp +++ b/clang/lib/CodeGen/Mangle.cpp @@ -62,14 +62,14 @@ bool CXXNameMangler::mangle(const NamedDecl *D) { // FIXME: Actually use a visitor to decode these? if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) { bool RequiresMangling = false; - // No mangled in an "implicit extern C" header. - if (Context.getSourceManager().getFileCharacteristic(FD->getLocation()) - == SrcMgr::C_ExternCSystem) - RequiresMangling = false; // Clang's "overloadable" attribute extension to C/C++ implies // name mangling (always). - else if (FD->getAttr<OverloadableAttr>()) + if (FD->getAttr<OverloadableAttr>()) RequiresMangling = true; + // No mangled in an "implicit extern C" header. + else if (Context.getSourceManager().getFileCharacteristic(FD->getLocation()) + == SrcMgr::C_ExternCSystem) + RequiresMangling = false; else if (Context.getLangOptions().CPlusPlus) { // C++ requires name mangling, unless we're in a C linkage // specification. diff --git a/clang/test/CodeGen/mangle.c b/clang/test/CodeGen/mangle.c new file mode 100644 index 00000000000..fbee4122d12 --- /dev/null +++ b/clang/test/CodeGen/mangle.c @@ -0,0 +1,9 @@ +// RUN: clang -arch i386 -emit-llvm -o %t %s && +// RUN: grep '@_Z2f0i' %t && +// RUN: grep '@_Z2f0l' %t + +// Make sure we mangle overloadable, even in C system headers. + +# 1 "somesystemheader.h" 1 3 4 +void __attribute__((__overloadable__)) f0(int a) {} +void __attribute__((__overloadable__)) f0(long b) {} |
