diff options
| author | Reid Kleckner <reid@kleckner.net> | 2014-09-23 00:00:14 +0000 |
|---|---|---|
| committer | Reid Kleckner <reid@kleckner.net> | 2014-09-23 00:00:14 +0000 |
| commit | 6c03130542028ada57b8a2872657be4252a65fa0 (patch) | |
| tree | 51eb91ddc906cbbd15adff8c9a0278ccb8041933 /clang/lib/CodeGen/ItaniumCXXABI.cpp | |
| parent | 0f79772ecccd3340c84b45f00144c7a5d5395cc8 (diff) | |
| download | bcm5719-llvm-6c03130542028ada57b8a2872657be4252a65fa0.tar.gz bcm5719-llvm-6c03130542028ada57b8a2872657be4252a65fa0.zip | |
Don't use comdats for initializers on platforms that don't support it
In particular, pre-.init_array ELF uses the .ctors section mechanism.
MinGW COFF also uses .ctors, now that I think about it. Therefore,
restrict this optimization to the two platforms that are currently known
to work: ELF with .init_array and COFF with .CRT$XCU.
llvm-svn: 218287
Diffstat (limited to 'clang/lib/CodeGen/ItaniumCXXABI.cpp')
| -rw-r--r-- | clang/lib/CodeGen/ItaniumCXXABI.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang/lib/CodeGen/ItaniumCXXABI.cpp b/clang/lib/CodeGen/ItaniumCXXABI.cpp index 84bfb6aa505..6d73a4b184c 100644 --- a/clang/lib/CodeGen/ItaniumCXXABI.cpp +++ b/clang/lib/CodeGen/ItaniumCXXABI.cpp @@ -1668,7 +1668,8 @@ void ItaniumCXXABI::EmitGuardedInit(CodeGenFunction &CGF, // The ABI says: It is suggested that it be emitted in the same COMDAT group // as the associated data object - if (!D.isLocalVarDecl() && var->isWeakForLinker() && CGM.supportsCOMDAT()) { + if (!D.isLocalVarDecl() && var->isWeakForLinker() && + CGM.supportsCOMDATInitializers()) { llvm::Comdat *C = CGM.getModule().getOrInsertComdat(var->getName()); guard->setComdat(C); var->setComdat(C); |
