[PATCH] D152090: [clang][Driver] Add -fcaret-diagnostics-max-lines as a driver option

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 5 21:39:31 PDT 2023


tbaeder added a comment.

In D152090#4395827 <https://reviews.llvm.org/D152090#4395827>, @MaskRay wrote:

> `def fcaret_diagnostics_max_lines` in `Options.td` has the `NoDriverOption` flag. Move it to other places with `BooleainFFlag` should work. 
> Then in `Clang.cpp` you can just write `Args.AddLastArg(...)`
>
> Can you add some description that this is related to D147875 <https://reviews.llvm.org/D147875> ?

Can you give a more concrete example? I modeled this patch after existing similar options, like `-fmacro-backtrace-limit`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152090



More information about the cfe-commits mailing list