[PATCH] D26857: [ELF] Print output name for command line errors

Eugene Leviant via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 09:17:47 PST 2016


evgeny777 added inline comments.


================
Comment at: ELF/Driver.cpp:99
     else
-      error("unknown emulation: " + Emul);
+      error(getOutputName() + ": unknown emulation: " + Emul);
   }
----------------
rafael wrote:
> I am not sure it makes sense to always print the output name. In here the issue has nothing to do with the files, it is just a broken -m option.
> 
> If you want a common format, maybe printing something like" <command line>:" in here would be better.
So do you propose to use **<command line>** as tag for all command line arguments errors?


Repository:
  rL LLVM

https://reviews.llvm.org/D26857





More information about the llvm-commits mailing list