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

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 09:03:22 PST 2016


rafael added inline comments.


================
Comment at: ELF/Driver.cpp:99
     else
-      error("unknown emulation: " + Emul);
+      error(getOutputName() + ": unknown emulation: " + Emul);
   }
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D26857





More information about the llvm-commits mailing list