[PATCH] D122039: [BOLT] Fix plt relocations symbol match

[Maksim Panchenko via Phabricator via llvm-commits]

maksfb added a comment.

Could you please rename test files to use "-" instead of "_"?



================
Comment at: bolt/test/AArch64/plt_gnu_ld.test:7
+// RUN: yaml2obj %p/Inputs/plt_gnu_ld.yaml &> %t.exe
+// RUN: llvm-bolt %t.bfd.exe -o %t.bolt.exe -use-old-text=0 -lite=0 \
+// RUN:   -print-cfg -print-only=main | FileCheck %s
----------------
I tried that fix locally and got "no PT_LOAD pheader seen" error.


================
Comment at: bolt/test/AArch64/plt_lld.test:4
+
+// RUN: %clang %cflags %p/../Inputs/plt.c -fuse-ld=lld \
+// RUN:    -o %t.lld.exe -Wl,-q
----------------
plt.c will not cross-compile since it includes header files.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122039