diff options
author | David Majnemer <david.majnemer@gmail.com> | 2014-04-29 06:18:53 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2014-04-29 06:18:53 +0000 |
commit | 846c2833ecaaffc0114282b77d3b5dacb44a01fe (patch) | |
tree | 8d25d538ab34e270e840723bf6de00ac03cd2827 /clang/lib/CodeGen | |
parent | 105d6abe506c7591118ce065bcbbea7399efd1f6 (diff) | |
download | bcm5719-llvm-846c2833ecaaffc0114282b77d3b5dacb44a01fe.tar.gz bcm5719-llvm-846c2833ecaaffc0114282b77d3b5dacb44a01fe.zip |
CodeGen: Reference temporaries inherit visibility
Reference temporaries inherited many properties from the variable that
they correspond to but visibility wasn't one of them.
llvm-svn: 207496
Diffstat (limited to 'clang/lib/CodeGen')
-rw-r--r-- | clang/lib/CodeGen/CodeGenModule.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index bed949db220..61039333008 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -2865,6 +2865,8 @@ llvm::Constant *CodeGenModule::GetAddrOfGlobalTemporary( // Create a global variable for this lifetime-extended temporary. llvm::GlobalValue::LinkageTypes Linkage = getLLVMLinkageVarDefinition(VD, Constant); + // There is no need for this temporary to have global linkage if the global + // variable has external linkage. if (Linkage == llvm::GlobalVariable::ExternalLinkage) Linkage = llvm::GlobalVariable::PrivateLinkage; unsigned AddrSpace = GetGlobalVarAddressSpace( @@ -2873,6 +2875,7 @@ llvm::Constant *CodeGenModule::GetAddrOfGlobalTemporary( getModule(), Type, Constant, Linkage, InitialValue, Name.c_str(), /*InsertBefore=*/nullptr, llvm::GlobalVariable::NotThreadLocal, AddrSpace); + setGlobalVisibility(GV, VD); GV->setAlignment( getContext().getTypeAlignInChars(MaterializedType).getQuantity()); if (VD->getTLSKind()) |