[llvm] [BOLT][AArch64] Add support for long absolute LLD thunks/veneers (PR #113408)

Paschalis Mpeis via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 09:18:21 PST 2024


================
@@ -29,30 +29,47 @@ static llvm::cl::opt<bool>
 namespace llvm {
 namespace bolt {
 
+static bool isPossibleVeneer(const BinaryFunction &BF) {
+  return BF.isAArch64Veneer() || BF.getOneName().starts_with("__AArch64");
----------------
paschalis-mpeis wrote:

Would it make sense to also check that this is not PI? eg:
```cpp
BF.isAArch64Veneer() || 
  (BF.getOneName().starts_with("__AArch64") && BF.getBinaryContext().HasFixedLoadAddress);
```

https://github.com/llvm/llvm-project/pull/113408


More information about the llvm-commits mailing list