[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