[PATCH] D55423: [LLD][ELF] - A fix for "linker script assignment loses relative nature of section" bug.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 11 02:20:19 PST 2018


grimar added a comment.

In D55423#1325166 <https://reviews.llvm.org/D55423#1325166>, @peter.smith wrote:

> I quite like the idea of renaming maybeNeedsThunks(), for example addAddressDependentContent()? We could make sure that there is always at least one pass of assignAddresses done there.


I decided to name it `finalizeAddressDependentContent` in D55550 <https://reviews.llvm.org/D55550> because `finalize` seems to be consistent prefix with the functions called around.


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

https://reviews.llvm.org/D55423





More information about the llvm-commits mailing list