Also don't inline dllimport functions referring to non-dllimport constructors.
The AST walker wasn't visiting CXXConstructExprs before. This is a follow-up to r281395. llvm-svn: 281413
This commit is contained in:
parent
45d56c2211
commit
1b3aee7ff9
|
@ -1693,6 +1693,8 @@ namespace {
|
|||
: public RecursiveASTVisitor<DLLImportFunctionVisitor> {
|
||||
bool SafeToInline = true;
|
||||
|
||||
bool shouldVisitImplicitCode() const { return true; }
|
||||
|
||||
bool VisitVarDecl(VarDecl *VD) {
|
||||
// A thread-local variable cannot be imported.
|
||||
SafeToInline = !VD->getTLSKind();
|
||||
|
@ -1708,6 +1710,10 @@ namespace {
|
|||
SafeToInline = !V->hasGlobalStorage() || V->hasAttr<DLLImportAttr>();
|
||||
return SafeToInline;
|
||||
}
|
||||
bool VisitCXXConstructExpr(CXXConstructExpr *E) {
|
||||
SafeToInline = E->getConstructor()->hasAttr<DLLImportAttr>();
|
||||
return SafeToInline;
|
||||
}
|
||||
bool VisitCXXDeleteExpr(CXXDeleteExpr *E) {
|
||||
SafeToInline = E->getOperatorDelete()->hasAttr<DLLImportAttr>();
|
||||
return SafeToInline;
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
// RUN: %clang_cc1 -triple i686-pc-win32 -fms-extensions -emit-llvm -O1 -disable-llvm-optzns %s -o - | FileCheck %s
|
||||
|
||||
class A {
|
||||
class __declspec(dllimport) A {
|
||||
virtual void m_fn1();
|
||||
};
|
||||
template <typename>
|
||||
|
@ -11,10 +11,10 @@ class __declspec(dllexport) C : B<int> {};
|
|||
|
||||
// CHECK-DAG: @[[VTABLE_C:.*]] = private unnamed_addr constant [2 x i8*] [i8* bitcast (%rtti.CompleteObjectLocator* @"\01??_R4C@@6B@" to i8*), i8* bitcast (void (%class.A*)* @"\01?m_fn1@A@@EAEXXZ" to i8*)]
|
||||
// CHECK-DAG: @[[VTABLE_B:.*]] = private unnamed_addr constant [2 x i8*] [i8* bitcast (%rtti.CompleteObjectLocator* @"\01??_R4?$B@H@@6B@" to i8*), i8* bitcast (void (%class.A*)* @"\01?m_fn1@A@@EAEXXZ" to i8*)], comdat($"\01??_S?$B@H@@6B@")
|
||||
// CHECK-DAG: @[[VTABLE_A:.*]] = private unnamed_addr constant [2 x i8*] [i8* bitcast (%rtti.CompleteObjectLocator* @"\01??_R4A@@6B@" to i8*), i8* bitcast (void (%class.A*)* @"\01?m_fn1@A@@EAEXXZ" to i8*)], comdat($"\01??_7A@@6B@")
|
||||
// CHECK-DAG: @[[VTABLE_A:.*]] = private unnamed_addr constant [2 x i8*] [i8* bitcast (%rtti.CompleteObjectLocator* @"\01??_R4A@@6B@" to i8*), i8* bitcast (void (%class.A*)* @"\01?m_fn1@A@@EAEXXZ" to i8*)], comdat($"\01??_SA@@6B@")
|
||||
|
||||
// CHECK-DAG: @"\01??_7C@@6B@" = dllexport unnamed_addr alias i8*, getelementptr inbounds ([2 x i8*], [2 x i8*]* @[[VTABLE_C]], i32 0, i32 1)
|
||||
// CHECK-DAG: @"\01??_S?$B@H@@6B@" = unnamed_addr alias i8*, getelementptr inbounds ([2 x i8*], [2 x i8*]* @[[VTABLE_B]], i32 0, i32 1)
|
||||
// CHECK-DAG: @"\01??_7A@@6B@" = unnamed_addr alias i8*, getelementptr inbounds ([2 x i8*], [2 x i8*]* @[[VTABLE_A]], i32 0, i32 1)
|
||||
// CHECK-DAG: @"\01??_SA@@6B@" = unnamed_addr alias i8*, getelementptr inbounds ([2 x i8*], [2 x i8*]* @[[VTABLE_A]], i32 0, i32 1)
|
||||
|
||||
// CHECK-DAG: @"\01??_8?$B@H@@7B@" = available_externally dllimport unnamed_addr constant [2 x i32] [i32 0, i32 4]
|
||||
|
|
|
@ -354,6 +354,10 @@ USECLASS(ClassWithNonDllImportField);
|
|||
USECLASS(ClassWithNonDllImportBase);
|
||||
// MO1-DAG: declare dllimport x86_thiscallcc void @"\01??1ClassWithNonDllImportBase@@QAE@XZ"(%struct.ClassWithNonDllImportBase*)
|
||||
// MO1-DAG: declare dllimport x86_thiscallcc void @"\01??1ClassWithNonDllImportField@@QAE@XZ"(%struct.ClassWithNonDllImportField*)
|
||||
struct ClassWithCtor { ClassWithCtor() {} };
|
||||
struct __declspec(dllimport) ClassWithNonDllImportFieldWithCtor { ClassWithCtor t; };
|
||||
USECLASS(ClassWithNonDllImportFieldWithCtor);
|
||||
// MO1-DAG: declare dllimport x86_thiscallcc %struct.ClassWithNonDllImportFieldWithCtor* @"\01??0ClassWithNonDllImportFieldWithCtor@@QAE@XZ"(%struct.ClassWithNonDllImportFieldWithCtor* returned)
|
||||
|
||||
// A dllimport function with a TLS variable must not be available_externally.
|
||||
__declspec(dllimport) inline void FunctionWithTLSVar() { static __thread int x = 42; }
|
||||
|
@ -665,7 +669,7 @@ namespace Vtordisp {
|
|||
// Don't dllimport the vtordisp.
|
||||
// MO1-DAG: define linkonce_odr x86_thiscallcc void @"\01?f@?$C@H@Vtordisp@@$4PPPPPPPM@A@AEXXZ"
|
||||
|
||||
class Base {
|
||||
class __declspec(dllimport) Base {
|
||||
virtual void f() {}
|
||||
};
|
||||
template <typename T>
|
||||
|
|
Loading…
Reference in New Issue