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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
