[PATCH] D71651: [llvm-readobj][test] - Refactor mips-st-other.test

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-readobj/ELF/mips-symbols-stother.test:4-5
+# RUN: yaml2obj %s > %t.o
+# RUN: llvm-readobj --symbols %t.o | FileCheck %s --strict-whitespace --check-prefix=MIPS-LLVM
+# RUN: llvm-readelf --symbols %t.o | FileCheck %s --strict-whitespace --check-prefix=MIPS-GNU
+
----------------
grimar wrote:
> jhenderson wrote:
> > Is --strict-whitespace important for this test? Do we anticipate spacing differences from other non-MIPS other bits?
> I think it is important for GNU. As output shows, the `Ndx` column is misplaced. Here I document this behavior.
> We might want to fix it. As far I can see we do not have a test that has sh_other > `0xF`
> and seems that is why this issue was never noticed. It is only a valid case for MIPS, hence I think it should be here.
> 
> For LLVM style perhaps we do not need -`-strict-whitespace`, I've added it just in case and for consistency.
Okay.


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

https://reviews.llvm.org/D71651