A C++ member function always has either weak linkage (if it's inline or defined inlin...

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71873 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/CodeGen/CodeGenModule.cpp b/lib/CodeGen/CodeGenModule.cpp
index 706a97354e47bb6ec2304306291512e9af052971..2c966150df204016cd469f646188aa1da62258f9 100644 (file)
--- a/lib/CodeGen/CodeGenModule.cpp
+++ b/lib/CodeGen/CodeGenModule.cpp
@@ -241,6 +241,14 @@ void CodeGenModule::EmitAnnotations() {
 
 static CodeGenModule::GVALinkage
 GetLinkageForFunction(const FunctionDecl *FD, const LangOptions &Features) {
+  if (const CXXMethodDecl *MD = dyn_cast<CXXMethodDecl>(FD)) {
+    // C++ member functions defined inside the class are always inline.
+    if (MD->isInline() || !MD->isOutOfLineDefinition())
+      return CodeGenModule::GVA_CXXInline;
+    
+    return CodeGenModule::GVA_StrongExternal;
+  }
+  
   // "static" functions get internal linkage.
   if (FD->getStorageClass() == FunctionDecl::Static)
     return CodeGenModule::GVA_Internal;

diff --git a/test/CodeGenCXX/member-functions.cpp b/test/CodeGenCXX/member-functions.cpp
index 84a18811d061081df1e229287bba06c65710df98..02c7e2c36d5faaaff850c7b619d33a39a8e3f3ad 100644 (file)
--- a/test/CodeGenCXX/member-functions.cpp
+++ b/test/CodeGenCXX/member-functions.cpp
@@ -26,14 +26,21 @@ struct S {
   inline ~S() { }
   
   
+  // RUN: grep "define linkonce_odr void @_ZN1S9f_inline1Ev" %t &&
   void f_inline1() { }
   // RUN: grep "define linkonce_odr void @_ZN1S9f_inline2Ev" %t &&
   inline void f_inline2() { }
   
-  // RUN: grep "define internal void @_ZN1S1gEv" %t
+  // RUN: grep "define linkonce_odr void @_ZN1S1gEv" %t &&
   static void g() { }
+  
+  static void f();
 };
 
+// RUN: grep "define void @_ZN1S1fEv" %t
+void S::f() {
+}
+
 void test2() {
   S s;