https://github.com/AaronBallman requested changes to this pull request.
I think using "broken" in the name of the diagnostic and the internal option is a bit problematic -- that's too much of a value judgement, IMO. After all, the previous functionality worked well enough for those folks. Would it make more sense to name it something along the lines of "retain substitution ast nodes"? https://github.com/llvm/llvm-project/pull/102510 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
