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

[Rui Ueyama via Phabricator via llvm-commits]

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:
> ruiu wrote:
> > 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.
> Hardcoding something is indeed easier. Problem is that here you depend on the naming rules.
> I.e. what if ".tmp." suffix changes to something else? Such tests will fail.
> FWIW in llvm tools test cases `-D` is used heavily.
Yeah, you are right, though I'm not too worried about that. If writing a full filename here is too much, I'd change this to `.o:(.foo)` so that it is not too picky about filenames.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68758