6b3351792c
For a definition (of most linkage types), dso_local is set for ELF -fno-pic/-fpie
and COFF, but not for Mach-O. This nuance causes unneeded binary format differences.
This patch replaces (function) `define ` with `define{{.*}} `,
(variable/constant/alias) `= ` with `={{.*}} `, or inserts appropriate `{{.*}} `
if there is an explicit linkage.
* Clang will set dso_local for Mach-O, which is currently implied by TargetMachine.cpp. This will make COFF/Mach-O and executable ELF similar.
* Eventually I hope we can make dso_local the textual LLVM IR default (write explicit "dso_preemptable" when applicable) and -fpic ELF will be similar to everything else. This patch helps move toward that goal.
39 lines
806 B
C++
39 lines
806 B
C++
// RUN: %clang_cc1 %s -triple=x86_64-apple-darwin10 -emit-llvm -o - -std=c++11 | FileCheck %s
|
|
// PR13424
|
|
|
|
namespace Test1 {
|
|
struct X {
|
|
virtual ~X(); // Key function.
|
|
virtual void f(); // Not a key function.
|
|
};
|
|
|
|
X::~X() = default;
|
|
|
|
// Verify that the vtable is emitted.
|
|
// CHECK-DAG: @_ZTVN5Test11XE ={{.*}} unnamed_addr constant
|
|
}
|
|
|
|
namespace Test2 {
|
|
struct X {
|
|
virtual ~X() = default; // Not a key function.
|
|
virtual void f(); // Key function.
|
|
};
|
|
|
|
void X::f() {}
|
|
|
|
// Verify that the vtable is emitted.
|
|
// CHECK-DAG: @_ZTVN5Test21XE ={{.*}} unnamed_addr constant
|
|
}
|
|
|
|
namespace Test3 {
|
|
struct X {
|
|
virtual ~X() = delete; // Not a key function.
|
|
virtual void f(); // Key function.
|
|
};
|
|
|
|
void X::f() {}
|
|
|
|
// Verify that the vtable is emitted.
|
|
// CHECK-DAG: @_ZTVN5Test31XE ={{.*}} unnamed_addr constant
|
|
}
|