[cfe-commits] r101948 - in /cfe/trunk: lib/CodeGen/Mangle.cpp test/CodeGenCXX/c-linkage.cpp

Fariborz Jahanian fjahanian at apple.com
Tue Apr 20 15:02:32 PDT 2010


Author: fjahanian
Date: Tue Apr 20 17:02:31 2010
New Revision: 101948

URL: http://llvm.org/viewvc/llvm-project?rev=101948&view=rev
Log:
Fixes a code gen. bug by removing an assert.
It is ok to have c++-ness inside extern "C" 
block. Fixes pr6644.

Added:
    cfe/trunk/test/CodeGenCXX/c-linkage.cpp
Modified:
    cfe/trunk/lib/CodeGen/Mangle.cpp

Modified: cfe/trunk/lib/CodeGen/Mangle.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/Mangle.cpp?rev=101948&r1=101947&r2=101948&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/Mangle.cpp (original)
+++ cfe/trunk/lib/CodeGen/Mangle.cpp Tue Apr 20 17:02:31 2010
@@ -306,8 +306,6 @@
 
 static const DeclContext *IgnoreLinkageSpecDecls(const DeclContext *DC) {
   while (isa<LinkageSpecDecl>(DC)) {
-    assert(cast<LinkageSpecDecl>(DC)->getLanguage() ==
-           LinkageSpecDecl::lang_cxx && "Unexpected linkage decl!");
     DC = DC->getParent();
   }
 

Added: cfe/trunk/test/CodeGenCXX/c-linkage.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/c-linkage.cpp?rev=101948&view=auto
==============================================================================
--- cfe/trunk/test/CodeGenCXX/c-linkage.cpp (added)
+++ cfe/trunk/test/CodeGenCXX/c-linkage.cpp Tue Apr 20 17:02:31 2010
@@ -0,0 +1,13 @@
+// RUN: %clang_cc1 -emit-llvm -o - %s | FileCheck %s
+// pr6644
+
+extern "C" {
+  namespace N {
+    struct X { 
+      virtual void f();
+    };
+    void X::f() { }
+  }
+}
+
+// CHECK: define void @_ZN1N1X1fEv





More information about the cfe-commits mailing list