================
@@ -102,12 +102,13 @@ class UncountedCallArgsChecker
         // if ((*P)->hasAttr<SafeRefCntblRawPtrAttr>())
         //  continue;
 
-        const auto *ArgType = (*P)->getType().getTypePtrOrNull();
-        if (!ArgType)
+        QualType ArgType = (*P)->getType().getCanonicalType();
+        const auto *TypePtr = ArgType.getTypePtrOrNull();
----------------
haoNoQ wrote:

`getTypePtrOrNull()` is unnecessary most of the time because you can do all the 
same operations on `QualType` directly, thanks to the overloaded 
`QualType::operator->()`.

I think `isUncountedPtr()` should simply accept a `QualType` directly.

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

Reply via email to