[lld] r185657 - Don't pass llvm::errs() all over the place. Diagnostics always go to stderr.

Craig Topper craig.topper at gmail.com
Thu Jul 4 20:31:58 PDT 2013


Don't the majority of clang's diagnostics go through the DiagnosticsEngine
which is itself controllable as to where it goes?


On Thu, Jul 4, 2013 at 9:43 PM, Rafael EspĂ­ndola <rafael.espindola at gmail.com
> wrote:

> On 4 July 2013 21:47, Nick Kledzik <kledzik at apple.com> wrote:
> > This seems worse off.  If lld was strictly a command line tool, I can
> see just just writing to stderr everywhere would be fine.  But we want lld
> to be a library too.  And the clients of the library will want better
> control over where diagnostics go.  In addition, if multiple things are
> being linked in parallel, the client could create a separate ostream for
> each parallel instance.  By forcing all instances to use the same (stderr)
> stream, the errors will be intermixed.
>
> It is a library, just like llvm and clang. Those are valid uses, but
> until we actually have someone using them, we should do like llvm and
> clang and have the simpler interface. I did grep llvm and clang and
> using llvm::errs() is by far the more common pattern.
>
> > -Nick
>
> Cheers,
> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130704/b42139a5/attachment.html>


More information about the llvm-commits mailing list