[PATCH] D25729: [ELF] Add "warning", "error" and "fatal error" prefixes to linker messages
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 18 11:20:30 PDT 2016
ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.
I'm a bit worried that showing both "error" and "fatal error" might confuse users. They are not different from the users' point of view. "Fatal error" will immediately exit, but "error" will also eventually exit with a failure code. Let's show "error:" for fatal. With that, LGTM.
More information about the llvm-commits