================
@@ -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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to