[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Mon May 13 11:12:56 PDT 2024


sam-mccall wrote:

All makes sense to me. 

I'd point out that the only revert I was asking for was asking for was of the deprecation of the flag to restore the old behavior, and *optionally* the default flip. 

The combination of {prominent oss library, accepted by clang-18 and gcc, now ICE, flag to fix is deprecated} doesn't seem tenable even if the code is invalid, so I don't think that resolving it hinged on reproducer quality.

Others requested a broader revert. I suspect this is fix-forward being bad for tree stability, and revert + partial-reapply being better in this respect. Which... well, tradeoffs.

In any case I think we lost some nuance here, I'll try to be more helpful in future!

https://github.com/llvm/llvm-project/pull/89807


More information about the cfe-commits mailing list