[PATCH] D42635: [ELF] - Report valid binary filename when reporting error.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 13:44:25 PST 2018


ruiu added inline comments.


================
Comment at: test/ELF/duplicated-synthetic-sym.s:1-7
 // RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
 // RUN: cd %S
 // RUN: not ld.lld %t.o --format=binary duplicated-synthetic-sym.s -o %t.elf 2>&1 | FileCheck %s
 // RUN: not ld.lld %t.o --format binary duplicated-synthetic-sym.s -o %t.elf 2>&1 | FileCheck %s
 
 // CHECK: duplicate symbol: _binary_duplicated_synthetic_sym_s_start
+// CHECK: defined at duplicated-synthetic-sym.s:(.data+0x0)
----------------
Can you change the test? Linking an .s file as a binary file is confusing. Just create a temporary file by doing something like this

  RUN: echo abc > %t.bin

and link that file.


https://reviews.llvm.org/D42635





More information about the llvm-commits mailing list