[PATCH] D125078: Implement a feature to show line numbers in diagnostics

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 9 10:04:57 PDT 2022


xbolva00 added a comment.

In D125078#3501061 <https://reviews.llvm.org/D125078#3501061>, @cjdb wrote:

> I'm in favour of this, but @rsmith warned me a little while ago that we mightn't be able to have stuff like this on by default because scripts probably depend on specific compiler output. Hyrum's Law <https://www.hyrumslaw.com/> strikes again :(
>
> (I personally think we should have it on by default and give a way to opt out, rather than opt in.)

I believe even more tools depend on gcc and gcc devs were able to turn it on by default, so kinda bad excuse, especially with opt out flag.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125078/new/

https://reviews.llvm.org/D125078



More information about the cfe-commits mailing list