[llvm-branch-commits] [ELF] Place .lbss/.lrodata/.ldata after .bss (PR #81224)

Peter Smith via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri Feb 9 08:26:37 PST 2024


================
@@ -1124,11 +1127,15 @@ template <class ELFT> void Writer<ELFT>::setReservedSymbolSections() {
   }
 
   if (last) {
-    // _edata points to the end of the last mapped initialized section.
+    // _edata points to the end of the last mapped initialized section before
----------------
smithp35 wrote:

Apart from the comment about .ldata may be after _edata, is this something we had wrong prior to this patch?

If so it could be worth splitting out into a separate patch so we can identify/test it separately. Ideally we'd want a test case for each of the conditions below. 

https://github.com/llvm/llvm-project/pull/81224


More information about the llvm-branch-commits mailing list