[PATCH] D58955: [WebAssembly] Remove trailing whitespaces in tests (NFC)

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 5 16:37:29 PST 2019


sbc100 accepted this revision.
sbc100 added a comment.
This revision is now accepted and ready to land.

Seems reasonable, although the trailing white space in the ll files (i imagine because I can't figure out how to see the the diff in phabricator)  is most likely due to the fact the yaml output of obj2yaml often includes trailing white space so when using copy and paste to include output check trailing whitespce is often introduced.    I'm OK with it being removed, but I'm also OK with trailing whitespace in the CHECK: lines if that whitespace is present in the output that is being checked.     I guess we should try to fix obj2yaml, although the beefit would be pretty tiny.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58955





More information about the llvm-commits mailing list