[PATCH] D100818: [lld-macho] Implement branch-range-extension thunks

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 30 07:26:10 PDT 2021


tschuett added a comment.

Feel free to ignore me, but it is an odd dance with the default implementation of populateThunk, which is unreachable.

And the following is only used by ARM64.

  size_t thunkSize = 0;
  uint64_t branchRange = 0;

Would a subclass of TargetInfo, like e.g. ThunkableTargetInfo with populateThunk, thunkSize, and branchRange be a nicer solution?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100818



More information about the llvm-commits mailing list