[PATCH] D120850: [BOLT] Handle ifuncs trampolines for aarch64

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 8 11:33:15 PST 2022


rafauler added inline comments.


================
Comment at: bolt/lib/Rewrite/RewriteInstance.cpp:1260
+
+  BinaryFunction *BF = BC->getBinaryFunctionAtAddress(EntryAddress);
+  if (BF && BC->isAArch64()) {
----------------
I don't understand why the BinaryFunction is already created for ifuncs? For X86, both ifuncs and regular PLTs are pretty similar, except for the relocation type.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120850



More information about the llvm-commits mailing list