[PATCH] D61666: [ELF] Optimize getISDThunkSec() to amortized O(1) in thunk creation

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 03:04:55 PDT 2019


MaskRay abandoned this revision.
MaskRay added a comment.

> if there is more than one branch relocation range.

Good point - there can be more than one branch types. I didn't know the code well when I created this patch.

If I choose 0x2000000 for ppc64le thunk section spacing (I tried several number slightly smaller than 0x2000000; they have no differences), `ThunkSections.size()` is at most 22. I'll abandon this.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D61666





More information about the llvm-commits mailing list