[PATCH] D34689: [LLD][ELF] Pre-create ThunkSections at Target specific intervals

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added inline comments.


================
Comment at: ELF/Relocations.cpp:1080
+          ISLimit = IS->OutSecOff + IS->getSize();
+          if (ISLimit >= ThunkUpperBound) {
+            addThunkSection(OS, ISR, PrevISLimit);
----------------
Should this be `>` instead of `>=`? `ISLimit` is one past the ending address of the current `IS`, and `ThunkUpperBound` is the maximum starting address of the thunk, so in the case where `ISLimit` == `ThunkUpperBound`, the thunk should be placed immediately after that `IS`, which is what a `>` comparison would accomplish.


https://reviews.llvm.org/D34689