[lld] [LLD][ELF] Allow memory region in OVERLAY (PR #133540)
Daniel Thornburgh via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 31 10:07:03 PDT 2025
mysterymath wrote:
> Thanks! Code looks great. Consider renaming lastInOverlay to firstInOverlay and moving the code to `expand*` to place overlay code together.
As written, this wouldn't be correct when the first overlay section is empty, as `expandOutputSection` would never be called. I do like `firstInOverlay` better though; it seems simpler to think about.
https://github.com/llvm/llvm-project/pull/133540
More information about the llvm-commits
mailing list