diff options
author | Derek Schuff <dschuff@google.com> | 2015-05-08 16:47:21 +0000 |
---|---|---|
committer | Derek Schuff <dschuff@google.com> | 2015-05-08 16:47:21 +0000 |
commit | 2312bd38114c932374f37aa4fe6c8fefc763dbfb (patch) | |
tree | a35e7f252d2bda095536bdf987bbaadc0188809c /clang/lib/CodeGen/CodeGenModule.h | |
parent | 049887b252f7f7cca78ade06f03075f0df8d247c (diff) | |
download | bcm5719-llvm-2312bd38114c932374f37aa4fe6c8fefc763dbfb.tar.gz bcm5719-llvm-2312bd38114c932374f37aa4fe6c8fefc763dbfb.zip |
Do not emit thunks with available_externally linkage in comdats
Functions with available_externally linkage will not be emitted to object
files (they will just be undefined symbols), so it does not make sense to
put them in comdats.
Creates a second overload of maybeSetTrivialComdat that uses the GlobalObject
instead of the Decl, and uses that in several places that had the faulty
logic.
Differential Revision: http://reviews.llvm.org/D9580
llvm-svn: 236879
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.h')
-rw-r--r-- | clang/lib/CodeGen/CodeGenModule.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/clang/lib/CodeGen/CodeGenModule.h b/clang/lib/CodeGen/CodeGenModule.h index feef6c2583e..6ce5c31cee9 100644 --- a/clang/lib/CodeGen/CodeGenModule.h +++ b/clang/lib/CodeGen/CodeGenModule.h @@ -608,6 +608,7 @@ public: const llvm::Triple &getTriple() const; bool supportsCOMDAT() const; void maybeSetTrivialComdat(const Decl &D, llvm::GlobalObject &GO); + void maybeSetTrivialComdat(llvm::GlobalObject &GO); CGCXXABI &getCXXABI() const { return *ABI; } llvm::LLVMContext &getLLVMContext() { return VMContext; } |