[PATCH] D64290: [tools] [llvm-nm] Default to reading from stdin not a.out
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 8 06:19:02 PDT 2019
jhenderson added a comment.
Is there a test case showing the new behaviour of reading from stdin now (both explicitly and implicitly)? We should add/extend it for the new (and existing) stdin behaviour.
================
Comment at: llvm/docs/CommandGuide/llvm-nm.rst:17
command line. Each symbol is listed along with some simple information about
-its provenance. If no filename is specified, *a.out* is used as the input.
-If *-* is used as a filename, :program:`llvm-nm` will read a file from its
-standard input stream.
+its provenance. If no filename is specified, or *-* is used as a filename,
+:program:`llvm-nm` will process a file on its standard input stream.
----------------
abrachet wrote:
> I have used the same language here to what existed prior to rL363065
Let's keep the phrase of "read from" rather than "process on", since llvm-nm doesn't do any processing (which implies major work or editing to me), just reading and interpreting.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64290/new/
https://reviews.llvm.org/D64290
More information about the llvm-commits
mailing list