[PATCH] D94472: [WIP][clang][cli] Command line round-trip for HeaderSearch options

Michael Spencer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 09:38:14 PST 2021


Bigcheese added a comment.

In D94472#2519838 <https://reviews.llvm.org/D94472#2519838>, @jansvoboda11 wrote:

> In D94472#2508018 <https://reviews.llvm.org/D94472#2508018>, @dexonsmith wrote:
>
>> `strict` mode additionally uses the `GeneratedArgs1` to fill CompilerInvocation, indirectly checking both directions by requiring tests to pass both with and without this round-trip. However, during development people generally only run the tests one way and the failure mode won't be ideal.
>
> So people build without assertions during development? In that case, I agree that erroring out on `GeneratedArgs1 != GeneratedArgs2` (in all kinds of builds) would improve the experience. I don't think there's anything preventing us to incorporate this into the current patch.

The only issue I have with this if always parsing twice has a noticeable performance impact for any users of Clang. Can we measure the impact? If it's small (< 100ms ?) then that's fine. I'm also concerned about users like cling and clangd.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94472



More information about the llvm-commits mailing list