[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