]> granicus.if.org Git - clang/commitdiff
Mangle extern "C" functions whose names are not simple identifiers.
authorRafael Espindola <rafael.espindola@gmail.com>
Thu, 14 Feb 2013 15:38:59 +0000 (15:38 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Thu, 14 Feb 2013 15:38:59 +0000 (15:38 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@175166 91177308-0d34-0410-b5e6-96231b3b80d8

lib/AST/ItaniumMangle.cpp
test/CodeGenCXX/c-linkage.cpp

index 1f95a2fee0c51407cd020e1e10d261f416b15e0b..d427b082a43221f99da98faa66070295a2f7f067 100644 (file)
@@ -373,8 +373,8 @@ bool ItaniumMangleContext::shouldMangleDeclName(const NamedDecl *D) {
     if (FD->hasAttr<OverloadableAttr>())
       return true;
 
-    // C functions and "main" are not mangled.
-    if (FD->isMain() || L == CLanguageLinkage)
+    // "main" is not mangled.
+    if (FD->isMain())
       return false;
 
     // C++ functions and those whose names are not a simple identifier need
@@ -382,6 +382,10 @@ bool ItaniumMangleContext::shouldMangleDeclName(const NamedDecl *D) {
     if (!FD->getDeclName().isIdentifier() || L == CXXLanguageLinkage)
       return true;
 
+    // C functions are not mangled.
+    if (L == CLanguageLinkage)
+      return false;
+
     // FIXME: Users assume they know the mangling of static functions
     // declared in extern "C" contexts, so we cannot always mangle them.
     // As an improvement, maybe we could mangle them only if they are actually
index c31ad438602085a3bfc2c350f875bf710e0886bd..cec9e282f69df861ab3134196b147b3bda66023d 100644 (file)
@@ -22,3 +22,10 @@ extern "C" {
     test2_f();
   }
 }
+
+extern "C" {
+  struct test3_s {
+  };
+  bool operator==(const int& a, const test3_s& b)  {
+  }
+}