[lld] r262840 - Adjust the .bss alignment to include common symbols.

Rafael Espindola via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 09:43:39 PST 2016


Author: rafael
Date: Mon Mar  7 11:43:39 2016
New Revision: 262840

URL: http://llvm.org/viewvc/llvm-project?rev=262840&view=rev
Log:
Adjust the .bss alignment to include common symbols.

Modified:
    lld/trunk/ELF/Writer.cpp
    lld/trunk/test/ELF/common.s

Modified: lld/trunk/ELF/Writer.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Writer.cpp?rev=262840&r1=262839&r2=262840&view=diff
==============================================================================
--- lld/trunk/ELF/Writer.cpp (original)
+++ lld/trunk/ELF/Writer.cpp Mon Mar  7 11:43:39 2016
@@ -722,6 +722,7 @@ void Writer<ELFT>::addCommonSymbols(std:
   uintX_t Off = getBss()->getSize();
   for (DefinedCommon *C : Syms) {
     Off = alignTo(Off, C->MaxAlignment);
+    Out<ELFT>::Bss->updateAlign(C->MaxAlignment);
     C->OffsetInBss = Off;
     Off += C->Size;
   }

Modified: lld/trunk/test/ELF/common.s
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/common.s?rev=262840&r1=262839&r2=262840&view=diff
==============================================================================
--- lld/trunk/test/ELF/common.s (original)
+++ lld/trunk/test/ELF/common.s Mon Mar  7 11:43:39 2016
@@ -13,6 +13,9 @@
 // CHECK-NEXT: Address: 0x11000
 // CHECK-NEXT: Offset:
 // CHECK-NEXT: Size: 22
+// CHECK-NEXT: Link: 0
+// CHECK-NEXT: Info: 0
+// CHECK-NEXT: AddressAlignment: 16
 
 // CHECK:      Name: sym1
 // CHECK-NEXT: Value: 0x11004




More information about the llvm-commits mailing list