<p dir="ltr">Lgtm! Thanks </p>
<div class="gmail_quote">On Sep 14, 2015 12:30 PM, "Davide Italiano" <<a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davide created this revision.<br>
davide added a reviewer: rafael.<br>
davide added a subscriber: llvm-commits.<br>
<br>
This is a follow up to r247518, addressing Rafael's post-commit reviews.<br>
As a general note, I think we could do a much better job testing for error conditions in tools. I already anticipated in a previous mail, but while implementing this I noticed that the code coverage we have for this is pretty low. I can arbitrarily remove error checking from various tests and the suite still passes.<br>
<br>
<a href="http://reviews.llvm.org/D12846" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12846</a><br>
<br>
Files:<br>
  test/tools/llvm-mc/basic.test<br>
  tools/llvm-mc/llvm-mc.cpp<br>
<br>
Index: tools/llvm-mc/llvm-mc.cpp<br>
===================================================================<br>
--- tools/llvm-mc/llvm-mc.cpp<br>
+++ tools/llvm-mc/llvm-mc.cpp<br>
@@ -395,15 +395,10 @@<br>
   // construct the Triple object.<br>
   Triple TheTriple(TripleName);<br>
<br>
-  if (InputFilename != "-" && !sys::fs::exists(InputFilename)) {<br>
-    errs() << ProgName << ": '" << InputFilename << "': "<br>
-           << "No such file\n";<br>
-    return 1;<br>
-  }<br>
   ErrorOr<std::unique_ptr<MemoryBuffer>> BufferPtr =<br>
       MemoryBuffer::getFileOrSTDIN(InputFilename);<br>
   if (std::error_code EC = BufferPtr.getError()) {<br>
-    errs() << ProgName << ": " << EC.message() << '\n';<br>
+    errs() << InputFilename << ": " << EC.message() << '\n';<br>
     return 1;<br>
   }<br>
   MemoryBuffer *Buffer = BufferPtr->get();<br>
Index: test/tools/llvm-mc/basic.test<br>
===================================================================<br>
--- test/tools/llvm-mc/basic.test<br>
+++ test/tools/llvm-mc/basic.test<br>
@@ -0,0 +1,2 @@<br>
+# RUN: not llvm-mc %t.blah -o %t2 2>&1 | FileCheck --check-prefix=ENOENT %s<br>
+# ENOENT: {{.*}}.blah: {{[Nn]}}o such file or directory<br>
<br>
<br>
</blockquote></div>