================
@@ -89,23 +89,20 @@ static bool SubstQualifier(Sema &SemaRef, const DeclT
*OldDecl, DeclT *NewDecl,
NestedNameSpecifierLoc NewQualifierLoc =
SemaRef.SubstNestedNameSpecifierLoc(OldDecl->getQualifierLoc(),
TemplateArgs);
-
- if (!NewQualifierLoc)
- return true;
-
+ assert(NewQualifierLoc &&
+ "substitution for the declaration qualifiers should never fail");
----------------
mizvekov wrote:
Not in a declarative nested name specifier, that has extra restrictions on top
of regular nested name specifiers.
With this patch, we enforce the declarative nested specifiers are well behaved,
so it's not possible to perform void_t SFINAE tricks that only affect
out-of-line definitions for example.
https://github.com/llvm/llvm-project/pull/190495
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
