diff options
| author | Daniel Dunbar <daniel@zuster.org> | 2011-06-28 23:33:55 +0000 |
|---|---|---|
| committer | Daniel Dunbar <daniel@zuster.org> | 2011-06-28 23:33:55 +0000 |
| commit | 421bb42c9fc7a786e8263bcff2f09afb4c468bcf (patch) | |
| tree | 614ad532d77176d1eab2be8a8244b24d43c4784e /clang/lib/Analysis/FormatString.cpp | |
| parent | dedd7d6ed6cc73d0cb1b2ddf3bea8a08262c7051 (diff) | |
| download | bcm5719-llvm-421bb42c9fc7a786e8263bcff2f09afb4c468bcf.tar.gz bcm5719-llvm-421bb42c9fc7a786e8263bcff2f09afb4c468bcf.zip | |
Revert r133024, "[format strings] correctly suggest correct type for '%@'
specifiers. Fixes <rdar://problem/9607158>." because it causes false positives
on some code that uses CF toll free bridging.
- I'll let Doug or Ted figure out the right fix here, possibly just to accept
any pointer type.
llvm-svn: 134041
Diffstat (limited to 'clang/lib/Analysis/FormatString.cpp')
| -rw-r--r-- | clang/lib/Analysis/FormatString.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Analysis/FormatString.cpp b/clang/lib/Analysis/FormatString.cpp index be214e0d86d..c1b5ea8a652 100644 --- a/clang/lib/Analysis/FormatString.cpp +++ b/clang/lib/Analysis/FormatString.cpp @@ -300,8 +300,7 @@ bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { argTy->isNullPtrType(); case ObjCPointerTy: - return argTy->getAs<ObjCObjectPointerType>() != NULL || - argTy->getAs<BlockPointerType>() != NULL; + return argTy->getAs<ObjCObjectPointerType>() != NULL; } // FIXME: Should be unreachable, but Clang is currently emitting |
