[PATCH] D154048: [PowerPC] fix jumptable encoding when ppc-use-absolute-jumptables is turned on
ChenZheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jul 2 22:14:42 PDT 2023
shchenz added inline comments.
================
Comment at: llvm/test/CodeGen/PowerPC/absol-jump-table-enabled.ll:73
; CHECK-A-PIC-LE-LABEL: .LJTI0_0:
-; CHECK-A-PIC-LE: .long .LBB0_6-.LJTI0_0
+; CHECK-A-PIC-LE: .quad .LBB0_6
;
----------------
This change seems problematic to me. PIC is a functionality option while -ppc-use-absolute-jumptables=true is an optimization option (save the add instruction?), so if -ppc-use-absolute-jumptables=true and --relocation-model=pic are both true, we should use relative jump table which matches PIC requirement.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154048/new/
https://reviews.llvm.org/D154048
More information about the llvm-commits
mailing list