[PATCH] D64290: [tools] [llvm-nm] Default to reading from stdin not a.out

Alex Brachet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 6 13:26:07 PDT 2019


abrachet marked an inline comment as done.
abrachet added a comment.

No tests tested if a.out was used as a default file so no need to change any other tests, they are all passing except the one that I added!



================
Comment at: llvm/test/tools/llvm-nm/warn-invalid-stdin.test:6
+
+# CHECK: llvm-nm: warning: can't read from keyboard input.
+# CHECK-NEXT: OVERVIEW: llvm symbol table dumper
----------------
This isn't working right now. FileCheck says the output was actually "The file was not recognized as a valid object file". Does llvm-lit redirect something to stdin?


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

https://reviews.llvm.org/D64290





More information about the llvm-commits mailing list