[PATCH] D72992: [llvm-objdump] - Add column headers for relocation printing

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 00:54:41 PST 2020


grimar added a comment.

Looking again, it seems it is OK to keep `--strict-whitespace` in `WebAssembly/relocations.test` as it is a generic test for relocations output for WASM.
I do not think we should add `--strict-whitespace` to COFF tests here as they are testing different things, not just regular output.
(i.e. it is probably not a proper place, they might want to add a generic test probably).
I wonder what other people thing about this though.


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

https://reviews.llvm.org/D72992





More information about the llvm-commits mailing list