summaryrefslogtreecommitdiffstats
path: root/clang/lib
diff options
context:
space:
mode:
authorTed Kremenek <kremenek@apple.com>2011-12-01 00:59:21 +0000
committerTed Kremenek <kremenek@apple.com>2011-12-01 00:59:21 +0000
commit764d63ad94e43dfa0939b5924b00c54fd07b7ff5 (patch)
tree660e5af2d770e6c98b270b5ec7c8d2baadd2b27a /clang/lib
parent85825aebc96ed4f84f5a1c7c141673d782a6d32b (diff)
downloadbcm5719-llvm-764d63ad94e43dfa0939b5924b00c54fd07b7ff5.tar.gz
bcm5719-llvm-764d63ad94e43dfa0939b5924b00c54fd07b7ff5.zip
Specially whitelist the selector 'addOperationWithBlock:' for the retain-cycle checking in -Warc-retain-cycles. This commonly
is hit by users using NSOperationQueue. Fixes <rdar://problem/10465721>. llvm-svn: 145548
Diffstat (limited to 'clang/lib')
-rw-r--r--clang/lib/Sema/SemaChecking.cpp8
1 files changed, 7 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp
index ec36963ed61..0d640a8c123 100644
--- a/clang/lib/Sema/SemaChecking.cpp
+++ b/clang/lib/Sema/SemaChecking.cpp
@@ -4533,8 +4533,14 @@ static bool isSetterLikeSelector(Selector sel) {
StringRef str = sel.getNameForSlot(0);
while (!str.empty() && str.front() == '_') str = str.substr(1);
- if (str.startswith("set") || str.startswith("add"))
+ if (str.startswith("set"))
str = str.substr(3);
+ else if (str.startswith("add")) {
+ // Specially whitelist 'addOperationWithBlock:'.
+ if (sel.getNumArgs() == 1 && str.startswith("addOperationWithBlock"))
+ return false;
+ str = str.substr(3);
+ }
else
return false;
OpenPOWER on IntegriCloud