================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to