[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