diff options
author | David Majnemer <david.majnemer@gmail.com> | 2016-02-10 17:40:47 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2016-02-10 17:40:47 +0000 |
commit | 1b2d0b4637a9b808aee88ba7a8b99fe446ed4c07 (patch) | |
tree | a9c04af507b8c2c9c1934a3a30be2c61c43b16be /clang/lib/AST/VTableBuilder.cpp | |
parent | 932b4cbf5d4b629c2796aeb9460b988a555a85e0 (diff) | |
download | bcm5719-llvm-1b2d0b4637a9b808aee88ba7a8b99fe446ed4c07.tar.gz bcm5719-llvm-1b2d0b4637a9b808aee88ba7a8b99fe446ed4c07.zip |
[MS ABI] Never reference dllimport'd vtables
Referencing a dllimported vtable is impossible in a constexpr
constructor. It would be friendlier to C++ programmers if we
synthesized a copy of the vftable which referenced imported virtual
functions. This would let us initialize the object in a way which
preserves both the intent to import functionality from another DLL while
also making constexpr work.
Differential Revision: http://reviews.llvm.org/D17061
llvm-svn: 260388
Diffstat (limited to 'clang/lib/AST/VTableBuilder.cpp')
-rw-r--r-- | clang/lib/AST/VTableBuilder.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/clang/lib/AST/VTableBuilder.cpp b/clang/lib/AST/VTableBuilder.cpp index bae018652f9..b728b279182 100644 --- a/clang/lib/AST/VTableBuilder.cpp +++ b/clang/lib/AST/VTableBuilder.cpp @@ -2548,7 +2548,6 @@ public: // Only include the RTTI component if we know that we will provide a // definition of the vftable. HasRTTIComponent = Context.getLangOpts().RTTIData && - !MostDerivedClass->hasAttr<DLLImportAttr>() && MostDerivedClass->getTemplateSpecializationKind() != TSK_ExplicitInstantiationDeclaration; |