diff options
| author | Duncan Sands <baldrick@free.fr> | 2008-05-26 19:58:59 +0000 |
|---|---|---|
| committer | Duncan Sands <baldrick@free.fr> | 2008-05-26 19:58:59 +0000 |
| commit | 28c3cff8250b3fe2adc6479306fe7dbdb48a1bdb (patch) | |
| tree | 7228a30f4d1d804257ae14016a3b4c254de30c06 /lib/Linker | |
| parent | 78ecf0d7b1ee7b1c44825d746b2f64e3a14aeb9b (diff) | |
| download | external_llvm-28c3cff8250b3fe2adc6479306fe7dbdb48a1bdb.zip external_llvm-28c3cff8250b3fe2adc6479306fe7dbdb48a1bdb.tar.gz external_llvm-28c3cff8250b3fe2adc6479306fe7dbdb48a1bdb.tar.bz2 | |
Factor code to copy global value attributes like
the section or the visibility from one global
value to another: copyAttributesFrom. This is
particularly useful for duplicating functions:
previously this was done by explicitly copying
each attribute in turn at each place where a
new function was created out of an old one, with
the result that obscure attributes were regularly
forgotten (like the collector or the section).
Hopefully now everything is uniform and nothing
is forgotten.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51567 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Linker')
| -rw-r--r-- | lib/Linker/LinkModules.cpp | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/lib/Linker/LinkModules.cpp b/lib/Linker/LinkModules.cpp index fae7d9d..66c68ca 100644 --- a/lib/Linker/LinkModules.cpp +++ b/lib/Linker/LinkModules.cpp @@ -351,20 +351,10 @@ static void ForceRenaming(GlobalValue *GV, const std::string &Name) { /// CopyGVAttributes - copy additional attributes (those not needed to construct /// a GlobalValue) from the SrcGV to the DestGV. static void CopyGVAttributes(GlobalValue *DestGV, const GlobalValue *SrcGV) { - // Propagate alignment, visibility and section info. - DestGV->setAlignment(std::max(DestGV->getAlignment(), SrcGV->getAlignment())); - DestGV->setSection(SrcGV->getSection()); - DestGV->setVisibility(SrcGV->getVisibility()); - if (const Function *SrcF = dyn_cast<Function>(SrcGV)) { - Function *DestF = cast<Function>(DestGV); - DestF->setCallingConv(SrcF->getCallingConv()); - DestF->setParamAttrs(SrcF->getParamAttrs()); - if (SrcF->hasCollector()) - DestF->setCollector(SrcF->getCollector()); - } else if (const GlobalVariable *SrcVar = dyn_cast<GlobalVariable>(SrcGV)) { - GlobalVariable *DestVar = cast<GlobalVariable>(DestGV); - DestVar->setThreadLocal(SrcVar->isThreadLocal()); - } + // Use the maximum alignment, rather than just copying the alignment of SrcGV. + unsigned Alignment = std::max(DestGV->getAlignment(), SrcGV->getAlignment()); + DestGV->copyAttributesFrom(SrcGV); + DestGV->setAlignment(Alignment); } /// GetLinkageResult - This analyzes the two global values and determines what |