[PATCH] D79103: Fix ELF/linkerscript/input-archive.s w/ @ in path

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 09:02:48 PDT 2020


thopre marked an inline comment as done.
thopre added inline comments.


================
Comment at: lld/test/ELF/linkerscript/thunk-gen-mips.s:18-20
 # CHECK-ANY: 00000000         .text           00000000 _start
 # CHECK-ANY: 0010000c l     F .text           00000010 __LA25Thunk_too_far
 # CHECK-ANY: 00100020 g     F .text           00000024 too_far
----------------
thopre wrote:
> CHECK-ANY is not a valid FileCheck directive and the FileCheck invocation uses the default prefix. This test is only testing the presence of SYMBOL TABLE in the output of llvm-objdump -t :-)
The test seems to have bitrotted as a result. This is the output of objdump -t I get when I run it manually:

SYMBOL TABLE:
00000001 l       *ABS*  00000000 MAIN
00000001 l       *ABS*  00000000 TARGET
00108050 l       .got   00000000 .hidden _gp
0010003c l     F .text  00000010 __LA25Thunk_too_far
00000030 g       .text  00000000 _start
00100050 g     F .text  0000000c too_far

So wrong address and flag for _start, wrong address and size for too_far.

I presume CHECK-ANY was meant to be CHECK-DAG and thus the order does not matter.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79103





More information about the llvm-commits mailing list