[PATCH] D68758: Improve error message for bad SHF_MERGE sections

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 10 03:01:38 PDT 2019


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


================
Comment at: lld/test/ELF/invalid/merge-invalid-size.s:4
 // RUN: not ld.lld %t.o -o /dev/null 2>&1 | FileCheck %s
-// CHECK: SHF_MERGE section size must be a multiple of sh_entsize
+// CHECK: merge-invalid-size.s.tmp.o:(.foo): SHF_MERGE section size (2) must be a multiple of sh_entsize (4)
 
----------------
grimar wrote:
> You should probably be able to do something like the following I think:
> 
> ```
> // ... | FileCheck %s - DFILE=%t.o
> // CHECK: [[FILE]]:(.foo): SHF_MERGE section size (2) must be a multiple of sh_entsize (4)
> ```
I'm not sure if that use of `-D`  makes things easier, so I'd like to stick with this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68758





More information about the llvm-commits mailing list