[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.


Repository:
  rL LLVM

https://reviews.llvm.org/D25729





More information about the llvm-commits mailing list