[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