[clang] [PATCH 1/4] [clang] Improve nested name specifier AST representation (PR #147835)

[via cfe-commits]

================
@@ -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