[PATCH] D116838: [ELF] -Map --why-extract=: print despite errors

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 22:59:30 PST 2022


ikudrin added inline comments.


================
Comment at: lld/ELF/LinkerScript.cpp:566-567
     error("discarding " + s.name + " section is not allowed");
+    if (&s == mainPart->relrDyn.get())
+      mainPart->relrDyn.reset();
+  }
----------------
There is no test that covers this change. And maybe it'll be better to fix the issue directly in `DynamicSection<ELFT>::computeContents()`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116838/new/

https://reviews.llvm.org/D116838



More information about the llvm-commits mailing list