================
@@ -2767,6 +2767,10 @@ class DependentBitIntTypeLoc final
     : public InheritingConcreteTypeLoc<TypeSpecTypeLoc, DependentBitIntTypeLoc,
                                        DependentBitIntType> {};
 
+class PredefinedSugarTypeLoc final
+    : public InheritingConcreteTypeLoc<TypeSpecTypeLoc, PredefinedSugarTypeLoc,
+                                       PredefinedSugarType> {};
----------------
mizvekov wrote:

Yeah, but for the current uses of PredefinedSugarType in this patch, we should 
never have to produce these TypeLocs, since this type is never written in 
source code.

It would be interesting to watch for this. I suspect that if we end up 
producing it, it would be because of some workaround somewhere, where some 
function expects a TypeLoc / TypeSourceInfo and we just produce a trivial one.

Likewise, serializing these TypeLocs seems also to be unnecessary.

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

Reply via email to