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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

In https://reviews.llvm.org/D44562#1043904, @espindola wrote:

> Using message() in all cases seems better.  We know the link is failing for unrelated reasons and we are just letting the user know why we are not printing better error messages (could not parse the debug info).


message() prints to stdout, which I hadn't realised earlier. This clearly should be printed to stderr, for which there is no mechanism currently, if we don't want a warning or error. Thinking about it more, the rest of LLVM has a concept of "remarks" which are "lower" severity messages than errors or warnings (and can't be promoted to errors, as I understand it). How about I add this to LLD? Alternatively, I'll need to add a note() function, or just print to errs() directly.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44562