alexander-shaposhnikov added inline comments.
================ Comment at: clang/lib/Sema/SemaConcept.cpp:781 + /*ForConstraintInstantiation=*/true, /*SkipForSpecialization*/ false); + Sema::SFINAETrap SFINAE(S, /*AccessCheckingSFINAE=*/false); + std::optional<Sema::CXXThisScopeRAII> ThisScope; ---------------- alexander-shaposhnikov wrote: > rsmith wrote: > > Just a small optimization: there's no point doing the transform if we have > > nothing to substitute. > > Just a small optimization: there's no point doing the transform if we have > > nothing to substitute. > > do not try to substitute empty MLTAL (suggested above by @rsmith) 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
