luken-google added a comment. There is logic in the template expansion that returns an error state if the template parameter pack was not at the end of the template parameter list. It returns `true` but does not generate any diagnostics, but the outer code error machinery assumes that diagnostics have already been generated and so considers the expression invalid and moves on.
As this fix no longer accepts these ill-formed templates as valid, that code no longer fires. I have strengthened the check there to an assert. PTAL. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138210/new/ https://reviews.llvm.org/D138210 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
