[PATCH] D44562: [ELF] Rework debug line parsing to use llvm::Error and callbacks (LLD-side)

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: Common/ErrorHandler.cpp:86-91
+void ErrorHandler::remark(const Twine &Msg) {
+  std::lock_guard<std::mutex> Lock(Mu);
+  newline(ErrorOS, Msg);
+  print("remark: ", raw_ostream::BLUE);
+  *ErrorOS << Msg << "\n";
+}
----------------
Honestly I don't think we want to add a new level of error messages, as I think error() for errors, warning() for warnings and message() for non-error messages is enough. If something needs to be fixed, it should be an error or a warning. If something doesn't have to be fixed, lld shouldn't print out anything. A situation like "verbose messages are printed out but you can ignore them" isn't actionable and thus not desirable.

So, can you choose either (1) show it as a warning or (2) don't show anything?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44562