rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land.
Does my horrible lambda example work now? If so, that seems like a useful testcase. ================ Comment at: clang/lib/Sema/SemaConcept.cpp:781 + /*ForConstraintInstantiation=*/true, /*SkipForSpecialization*/ false); + Sema::SFINAETrap SFINAE(S, /*AccessCheckingSFINAE=*/false); + std::optional<Sema::CXXThisScopeRAII> ThisScope; ---------------- Just a small optimization: there's no point doing the transform if we have nothing to substitute. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146178/new/ https://reviews.llvm.org/D146178 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
