[PATCH] D159513: [Bolt] fix a relocation bug for R_AARCH64_CALL26
Sinan Lin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 18 01:33:25 PDT 2023
sinan updated this revision to Diff 556932.
sinan added a comment.
update the test case.
1. check address order with llvm-nm --numeric-sort;
2. check the message of fail to patch entries;
3. add comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159513/new/
https://reviews.llvm.org/D159513
Files:
bolt/lib/Core/Relocation.cpp
bolt/test/AArch64/reloc-call26.s
Index: bolt/test/AArch64/reloc-call26.s
===================================================================
--- bolt/test/AArch64/reloc-call26.s
+++ bolt/test/AArch64/reloc-call26.s
@@ -1,29 +1,51 @@
## This test checks processing of R_AARCH64_CALL26 relocation
## when option `--funcs` is enabled
+## We want to test on relocations against functions with both higher
+## and lower addresses. The '--force-patch' option is used to prevent
+## the functions func1 and func2 from being optimized, so that their
+## addresses can remain unchanged. Therefore, the relocations can be
+## updated via encodeValueAArch64 and the address order in the output
+## binary is func1 < _start < func2.
+
# REQUIRES: system-linux
# RUN: llvm-mc -filetype=obj -triple aarch64-unknown-unknown \
# RUN: %s -o %t.o
# RUN: %clang %cflags %t.o -o %t.exe -Wl,-q
-# RUN: llvm-bolt %t.exe -o %t.bolt --funcs=func1
+# RUN: llvm-bolt %t.exe -o %t.bolt --funcs=func1,func2 \
+# RUN: --force-patch 2>&1 | FileCheck %s -check-prefix=CHECK-BOLT
# RUN: llvm-objdump -d --disassemble-symbols='_start' %t.bolt | \
# RUN: FileCheck %s
+# RUN: llvm-nm --numeric-sort --extern-only %t.bolt | FileCheck \
+# RUN: %s -check-prefix=CHECK-FUNC-ORDER
+# CHECK-BOLT: BOLT-WARNING: failed to patch entries in func1. The function will not be optimized.
+# CHECK-BOLT: BOLT-WARNING: failed to patch entries in func2. The function will not be optimized.
# CHECK: {{.*}} bl {{.*}} <func1>
+# CHECK: {{.*}} bl {{.*}} <func2>
+
+# CHECK-FUNC-ORDER: {{.*}} func1
+# CHECK-FUNC-ORDER-NEXT: {{.*}} _start
+# CHECK-FUNC-ORDER-NEXT: {{.*}} func2
.text
.align 4
+ .global func1
+ .type func1, %function
+func1:
+ ret
+ .size func1, .-func1
.global _start
.type _start, %function
_start:
bl func1
+ bl func2
mov w8, #93
svc #0
.size _start, .-_start
-
- .global func1
- .type func1, %function
-func1:
+ .global func2
+ .type func2, %function
+func2:
ret
- .size func1, .-func1
+ .size func2, .-func2
Index: bolt/lib/Core/Relocation.cpp
===================================================================
--- bolt/lib/Core/Relocation.cpp
+++ bolt/lib/Core/Relocation.cpp
@@ -352,7 +352,7 @@
assert(isInt<28>(Value) && "only PC +/- 128MB is allowed for direct call");
// Immediate goes in bits 25:0 of BL.
// OP 1001_01 goes in bits 31:26 of BL.
- Value = (Value >> 2) | 0x94000000ULL;
+ Value = ((Value >> 2) & 0x3ffffff) | 0x94000000ULL;
break;
}
return Value;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D159513.556932.patch
Type: text/x-patch
Size: 2510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230918/02b88fc6/attachment.bin>
More information about the llvm-commits
mailing list