diff options
| author | Dale Johannesen <dalej@apple.com> | 2008-05-14 20:12:51 +0000 |
|---|---|---|
| committer | Dale Johannesen <dalej@apple.com> | 2008-05-14 20:12:51 +0000 |
| commit | 49c441222d19a0ae12ef357ec90d82da89e08a94 (patch) | |
| tree | e7d1caf5f9d6c359802174c4a01644b9aa326202 /lib/CodeGen/ELFWriter.cpp | |
| parent | aaa364e68d32387a9bbe7c6ab5f16c273f23e68c (diff) | |
| download | external_llvm-49c441222d19a0ae12ef357ec90d82da89e08a94.zip external_llvm-49c441222d19a0ae12ef357ec90d82da89e08a94.tar.gz external_llvm-49c441222d19a0ae12ef357ec90d82da89e08a94.tar.bz2 | |
Add CommonLinkage; currently tentative definitions
are represented as "weak", but there are subtle differences
in some cases on Darwin, so we need both. The intent
is that "common" will behave identically to "weak" unless
somebody changes their target to do something else.
No functional change as yet.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51118 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/ELFWriter.cpp')
| -rw-r--r-- | lib/CodeGen/ELFWriter.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/CodeGen/ELFWriter.cpp b/lib/CodeGen/ELFWriter.cpp index 63fee3a..27f23f4 100644 --- a/lib/CodeGen/ELFWriter.cpp +++ b/lib/CodeGen/ELFWriter.cpp @@ -282,7 +282,8 @@ void ELFWriter::EmitGlobal(GlobalVariable *GV) { // If this global is part of the common block, add it now. Variables are // part of the common block if they are zero initialized and allowed to be // merged with other symbols. - if (GV->hasLinkOnceLinkage() || GV->hasWeakLinkage()) { + if (GV->hasLinkOnceLinkage() || GV->hasWeakLinkage() || + GV->hasCommonLinkage()) { ELFSym CommonSym(GV); // Value for common symbols is the alignment required. CommonSym.Value = Align; @@ -313,7 +314,7 @@ void ELFWriter::EmitGlobal(GlobalVariable *GV) { BSSSym.SetType(ELFSym::STT_OBJECT); switch (GV->getLinkage()) { - default: // weak/linkonce handled above + default: // weak/linkonce/common handled above assert(0 && "Unexpected linkage type!"); case GlobalValue::AppendingLinkage: // FIXME: This should be improved! case GlobalValue::ExternalLinkage: |