[PATCH] D110217: [ELF][AArch64] Refine and fix the condition when BTI/PAC PLT needs bti c

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 13:44:51 PDT 2022


MaskRay added inline comments.


================
Comment at: lld/test/ELF/aarch64-ifunc-bti.s:10
+# RUN: ld.lld -shared -Bsymbolic %t1.so %t.o -o %t.so
+# RUN: llvm-objdump -d --no-show-raw-insn --mattr=+bti %t | FileCheck %s --check-prefix=SHARED
+
----------------
abrachet wrote:
> I've just noticed this while doing something unrelated to this patch, but I think that this should be %t.so? Looks like it'll just work with that change
Thanks for spotting this! I'll fix it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110217



More information about the llvm-commits mailing list