dcci wrote: > You should also add test coverage (clang/test/SemaCXX) for the changes and a > release note (clang/docs/ReleaseNotes.rst).
@AaronBallman -- are you OK with a crash-on-invalid (the one in first post)? The original example we had crashed on valid code, but reducing a 35MB file with a lot of SFINAE is taking weeks with c-reduce. Thanks for your review :) https://github.com/llvm/llvm-project/pull/98102 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
