================
@@ -2068,7 +2080,9 @@ class ASTContext : public RefCountedBase<ASTContext> {
   /// if it hasn't yet been built.
   QualType getRawCFConstantStringType() const {
     if (CFConstantStringTypeDecl)
-      return getTypedefType(CFConstantStringTypeDecl);
+      return getTypedefType(ElaboratedTypeKeyword::None,
+                            /*Qualifier=*/std::nullopt,
+                            CFConstantStringTypeDecl);
----------------
Sirraide wrote:

Thoughts about moving that third parameter to the front and adding default 
argument for what is currently the 1st and 2nd parameter? Because I’m noticing 
that there are a bunch of call sites that pass `ElaboratedTypeKeyword::None, 
/*Qualifier=*/std::nullopt` as the first two params.

https://github.com/llvm/llvm-project/pull/147835
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to