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