Benjamin Kramer | d81108f | 2012-11-14 15:08:31 +0000 | [diff] [blame] | 1 | //===--- TransUnbridgedCasts.cpp - Transformations to ARC mode ------------===// |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // rewriteUnbridgedCasts: |
| 11 | // |
| 12 | // A cast of non-objc pointer to an objc one is checked. If the non-objc pointer |
| 13 | // is from a file-level variable, __bridge cast is used to convert it. |
| 14 | // For the result of a function call that we know is +1/+0, |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 15 | // __bridge/CFBridgingRelease is used. |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 16 | // |
| 17 | // NSString *str = (NSString *)kUTTypePlainText; |
| 18 | // str = b ? kUTTypeRTF : kUTTypePlainText; |
| 19 | // NSString *_uuidString = (NSString *)CFUUIDCreateString(kCFAllocatorDefault, |
| 20 | // _uuid); |
| 21 | // ----> |
| 22 | // NSString *str = (__bridge NSString *)kUTTypePlainText; |
| 23 | // str = (__bridge NSString *)(b ? kUTTypeRTF : kUTTypePlainText); |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 24 | // NSString *_uuidString = (NSString *) |
| 25 | // CFBridgingRelease(CFUUIDCreateString(kCFAllocatorDefault, _uuid)); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 26 | // |
| 27 | // For a C pointer to ObjC, for casting 'self', __bridge is used. |
| 28 | // |
| 29 | // CFStringRef str = (CFStringRef)self; |
| 30 | // ----> |
| 31 | // CFStringRef str = (__bridge CFStringRef)self; |
| 32 | // |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 33 | // Uses of Block_copy/Block_release macros are rewritten: |
| 34 | // |
| 35 | // c = Block_copy(b); |
| 36 | // Block_release(c); |
| 37 | // ----> |
| 38 | // c = [b copy]; |
| 39 | // <removed> |
| 40 | // |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 41 | //===----------------------------------------------------------------------===// |
| 42 | |
| 43 | #include "Transforms.h" |
| 44 | #include "Internals.h" |
Benjamin Kramer | 4ab984e | 2012-07-04 20:19:54 +0000 | [diff] [blame] | 45 | #include "clang/AST/ASTContext.h" |
Benjamin Kramer | ea70eb3 | 2012-12-01 15:09:41 +0000 | [diff] [blame] | 46 | #include "clang/AST/Attr.h" |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 47 | #include "clang/AST/ParentMap.h" |
Benjamin Kramer | ea70eb3 | 2012-12-01 15:09:41 +0000 | [diff] [blame] | 48 | #include "clang/Analysis/DomainSpecific/CocoaConventions.h" |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 49 | #include "clang/Basic/SourceManager.h" |
Benjamin Kramer | 4ab984e | 2012-07-04 20:19:54 +0000 | [diff] [blame] | 50 | #include "clang/Lex/Lexer.h" |
| 51 | #include "clang/Sema/SemaDiagnostic.h" |
Benjamin Kramer | 4903802 | 2012-02-04 13:45:25 +0000 | [diff] [blame] | 52 | #include "llvm/ADT/SmallString.h" |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 53 | |
| 54 | using namespace clang; |
| 55 | using namespace arcmt; |
| 56 | using namespace trans; |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 57 | |
| 58 | namespace { |
| 59 | |
| 60 | class UnbridgedCastRewriter : public RecursiveASTVisitor<UnbridgedCastRewriter>{ |
| 61 | MigrationPass &Pass; |
| 62 | IdentifierInfo *SelfII; |
Ahmed Charles | b898432 | 2014-03-07 20:03:18 +0000 | [diff] [blame] | 63 | std::unique_ptr<ParentMap> StmtMap; |
Argyrios Kyrtzidis | b03cc79 | 2012-06-07 00:44:06 +0000 | [diff] [blame] | 64 | Decl *ParentD; |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 65 | Stmt *Body; |
Ahmed Charles | b898432 | 2014-03-07 20:03:18 +0000 | [diff] [blame] | 66 | mutable std::unique_ptr<ExprSet> Removables; |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 67 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 68 | public: |
Craig Topper | 8ae1203 | 2014-05-07 06:21:57 +0000 | [diff] [blame] | 69 | UnbridgedCastRewriter(MigrationPass &pass) |
| 70 | : Pass(pass), ParentD(nullptr), Body(nullptr) { |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 71 | SelfII = &Pass.Ctx.Idents.get("self"); |
| 72 | } |
| 73 | |
Argyrios Kyrtzidis | b03cc79 | 2012-06-07 00:44:06 +0000 | [diff] [blame] | 74 | void transformBody(Stmt *body, Decl *ParentD) { |
| 75 | this->ParentD = ParentD; |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 76 | Body = body; |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 77 | StmtMap.reset(new ParentMap(body)); |
| 78 | TraverseStmt(body); |
| 79 | } |
| 80 | |
Argyrios Kyrtzidis | 551c83d | 2013-08-09 20:20:25 +0000 | [diff] [blame] | 81 | bool TraverseBlockDecl(BlockDecl *D) { |
| 82 | // ParentMap does not enter into a BlockDecl to record its stmts, so use a |
| 83 | // new UnbridgedCastRewriter to handle the block. |
| 84 | UnbridgedCastRewriter(Pass).transformBody(D->getBody(), D); |
| 85 | return true; |
| 86 | } |
| 87 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 88 | bool VisitCastExpr(CastExpr *E) { |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 89 | if (E->getCastKind() != CK_CPointerToObjCPointerCast && |
| 90 | E->getCastKind() != CK_BitCast && |
| 91 | E->getCastKind() != CK_AnyPointerToBlockPointerCast) |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 92 | return true; |
| 93 | |
| 94 | QualType castType = E->getType(); |
| 95 | Expr *castExpr = E->getSubExpr(); |
| 96 | QualType castExprType = castExpr->getType(); |
| 97 | |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 98 | if (castType->isObjCRetainableType() == castExprType->isObjCRetainableType()) |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 99 | return true; |
| 100 | |
| 101 | bool exprRetainable = castExprType->isObjCIndirectLifetimeType(); |
| 102 | bool castRetainable = castType->isObjCIndirectLifetimeType(); |
| 103 | if (exprRetainable == castRetainable) return true; |
| 104 | |
| 105 | if (castExpr->isNullPointerConstant(Pass.Ctx, |
| 106 | Expr::NPC_ValueDependentIsNull)) |
| 107 | return true; |
| 108 | |
| 109 | SourceLocation loc = castExpr->getExprLoc(); |
| 110 | if (loc.isValid() && Pass.Ctx.getSourceManager().isInSystemHeader(loc)) |
| 111 | return true; |
| 112 | |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 113 | if (castType->isObjCRetainableType()) |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 114 | transformNonObjCToObjCCast(E); |
| 115 | else |
| 116 | transformObjCToNonObjCCast(E); |
| 117 | |
| 118 | return true; |
| 119 | } |
| 120 | |
| 121 | private: |
| 122 | void transformNonObjCToObjCCast(CastExpr *E) { |
| 123 | if (!E) return; |
| 124 | |
| 125 | // Global vars are assumed that are cast as unretained. |
| 126 | if (isGlobalVar(E)) |
| 127 | if (E->getSubExpr()->getType()->isPointerType()) { |
| 128 | castToObjCObject(E, /*retained=*/false); |
| 129 | return; |
| 130 | } |
| 131 | |
| 132 | // If the cast is directly over the result of a Core Foundation function |
| 133 | // try to figure out whether it should be cast as retained or unretained. |
| 134 | Expr *inner = E->IgnoreParenCasts(); |
| 135 | if (CallExpr *callE = dyn_cast<CallExpr>(inner)) { |
| 136 | if (FunctionDecl *FD = callE->getDirectCallee()) { |
Aaron Ballman | 9ead124 | 2013-12-19 02:39:40 +0000 | [diff] [blame] | 137 | if (FD->hasAttr<CFReturnsRetainedAttr>()) { |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 138 | castToObjCObject(E, /*retained=*/true); |
| 139 | return; |
| 140 | } |
Aaron Ballman | 9ead124 | 2013-12-19 02:39:40 +0000 | [diff] [blame] | 141 | if (FD->hasAttr<CFReturnsNotRetainedAttr>()) { |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 142 | castToObjCObject(E, /*retained=*/false); |
| 143 | return; |
| 144 | } |
| 145 | if (FD->isGlobal() && |
| 146 | FD->getIdentifier() && |
| 147 | ento::cocoa::isRefType(E->getSubExpr()->getType(), "CF", |
| 148 | FD->getIdentifier()->getName())) { |
| 149 | StringRef fname = FD->getIdentifier()->getName(); |
| 150 | if (fname.endswith("Retain") || |
| 151 | fname.find("Create") != StringRef::npos || |
| 152 | fname.find("Copy") != StringRef::npos) { |
Fariborz Jahanian | eb00284 | 2012-01-31 21:58:23 +0000 | [diff] [blame] | 153 | // Do not migrate to couple of bridge transfer casts which |
| 154 | // cancel each other out. Leave it unchanged so error gets user |
| 155 | // attention instead. |
| 156 | if (FD->getName() == "CFRetain" && |
| 157 | FD->getNumParams() == 1 && |
| 158 | FD->getParent()->isTranslationUnit() && |
Rafael Espindola | 3ae0005 | 2013-05-13 00:12:11 +0000 | [diff] [blame] | 159 | FD->isExternallyVisible()) { |
Fariborz Jahanian | eb00284 | 2012-01-31 21:58:23 +0000 | [diff] [blame] | 160 | Expr *Arg = callE->getArg(0); |
Fariborz Jahanian | a1c1b15 | 2012-01-31 22:09:44 +0000 | [diff] [blame] | 161 | if (const ImplicitCastExpr *ICE = dyn_cast<ImplicitCastExpr>(Arg)) { |
Fariborz Jahanian | eb00284 | 2012-01-31 21:58:23 +0000 | [diff] [blame] | 162 | const Expr *sub = ICE->getSubExpr(); |
| 163 | QualType T = sub->getType(); |
| 164 | if (T->isObjCObjectPointerType()) |
| 165 | return; |
| 166 | } |
| 167 | } |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 168 | castToObjCObject(E, /*retained=*/true); |
| 169 | return; |
| 170 | } |
| 171 | |
| 172 | if (fname.find("Get") != StringRef::npos) { |
| 173 | castToObjCObject(E, /*retained=*/false); |
| 174 | return; |
| 175 | } |
| 176 | } |
| 177 | } |
| 178 | } |
Argyrios Kyrtzidis | b03cc79 | 2012-06-07 00:44:06 +0000 | [diff] [blame] | 179 | |
| 180 | // If returning an ivar or a member of an ivar from a +0 method, use |
| 181 | // a __bridge cast. |
| 182 | Expr *base = inner->IgnoreParenImpCasts(); |
| 183 | while (isa<MemberExpr>(base)) |
| 184 | base = cast<MemberExpr>(base)->getBase()->IgnoreParenImpCasts(); |
| 185 | if (isa<ObjCIvarRefExpr>(base) && |
| 186 | isa<ReturnStmt>(StmtMap->getParentIgnoreParenCasts(E))) { |
| 187 | if (ObjCMethodDecl *method = dyn_cast_or_null<ObjCMethodDecl>(ParentD)) { |
| 188 | if (!method->hasAttr<NSReturnsRetainedAttr>()) { |
| 189 | castToObjCObject(E, /*retained=*/false); |
| 190 | return; |
| 191 | } |
| 192 | } |
| 193 | } |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 194 | } |
| 195 | |
| 196 | void castToObjCObject(CastExpr *E, bool retained) { |
| 197 | rewriteToBridgedCast(E, retained ? OBC_BridgeTransfer : OBC_Bridge); |
| 198 | } |
| 199 | |
| 200 | void rewriteToBridgedCast(CastExpr *E, ObjCBridgeCastKind Kind) { |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 201 | Transaction Trans(Pass.TA); |
| 202 | rewriteToBridgedCast(E, Kind, Trans); |
| 203 | } |
| 204 | |
| 205 | void rewriteToBridgedCast(CastExpr *E, ObjCBridgeCastKind Kind, |
| 206 | Transaction &Trans) { |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 207 | TransformActions &TA = Pass.TA; |
| 208 | |
| 209 | // We will remove the compiler diagnostic. |
| 210 | if (!TA.hasDiagnostic(diag::err_arc_mismatched_cast, |
| 211 | diag::err_arc_cast_requires_bridge, |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 212 | E->getLocStart())) { |
| 213 | Trans.abort(); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 214 | return; |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 215 | } |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 216 | |
| 217 | StringRef bridge; |
| 218 | switch(Kind) { |
| 219 | case OBC_Bridge: |
| 220 | bridge = "__bridge "; break; |
| 221 | case OBC_BridgeTransfer: |
| 222 | bridge = "__bridge_transfer "; break; |
| 223 | case OBC_BridgeRetained: |
| 224 | bridge = "__bridge_retained "; break; |
| 225 | } |
| 226 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 227 | TA.clearDiagnostic(diag::err_arc_mismatched_cast, |
| 228 | diag::err_arc_cast_requires_bridge, |
| 229 | E->getLocStart()); |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 230 | if (Kind == OBC_Bridge || !Pass.CFBridgingFunctionsDefined()) { |
| 231 | if (CStyleCastExpr *CCE = dyn_cast<CStyleCastExpr>(E)) { |
| 232 | TA.insertAfterToken(CCE->getLParenLoc(), bridge); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 233 | } else { |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 234 | SourceLocation insertLoc = E->getSubExpr()->getLocStart(); |
| 235 | SmallString<128> newCast; |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 236 | newCast += '('; |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 237 | newCast += bridge; |
| 238 | newCast += E->getType().getAsString(Pass.Ctx.getPrintingPolicy()); |
| 239 | newCast += ')'; |
| 240 | |
| 241 | if (isa<ParenExpr>(E->getSubExpr())) { |
| 242 | TA.insert(insertLoc, newCast.str()); |
| 243 | } else { |
| 244 | newCast += '('; |
| 245 | TA.insert(insertLoc, newCast.str()); |
| 246 | TA.insertAfterToken(E->getLocEnd(), ")"); |
| 247 | } |
| 248 | } |
| 249 | } else { |
| 250 | assert(Kind == OBC_BridgeTransfer || Kind == OBC_BridgeRetained); |
Jordan Rose | 288c421 | 2012-06-07 01:10:31 +0000 | [diff] [blame] | 251 | SmallString<32> BridgeCall; |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 252 | |
| 253 | Expr *WrapE = E->getSubExpr(); |
Jordan Rose | 288c421 | 2012-06-07 01:10:31 +0000 | [diff] [blame] | 254 | SourceLocation InsertLoc = WrapE->getLocStart(); |
| 255 | |
| 256 | SourceManager &SM = Pass.Ctx.getSourceManager(); |
| 257 | char PrevChar = *SM.getCharacterData(InsertLoc.getLocWithOffset(-1)); |
| 258 | if (Lexer::isIdentifierBodyChar(PrevChar, Pass.Ctx.getLangOpts())) |
| 259 | BridgeCall += ' '; |
| 260 | |
| 261 | if (Kind == OBC_BridgeTransfer) |
| 262 | BridgeCall += "CFBridgingRelease"; |
| 263 | else |
| 264 | BridgeCall += "CFBridgingRetain"; |
| 265 | |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 266 | if (isa<ParenExpr>(WrapE)) { |
Jordan Rose | 288c421 | 2012-06-07 01:10:31 +0000 | [diff] [blame] | 267 | TA.insert(InsertLoc, BridgeCall); |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 268 | } else { |
Jordan Rose | 288c421 | 2012-06-07 01:10:31 +0000 | [diff] [blame] | 269 | BridgeCall += '('; |
| 270 | TA.insert(InsertLoc, BridgeCall); |
Argyrios Kyrtzidis | 273c7c4 | 2012-06-01 00:10:47 +0000 | [diff] [blame] | 271 | TA.insertAfterToken(WrapE->getLocEnd(), ")"); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 272 | } |
| 273 | } |
| 274 | } |
| 275 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 276 | void rewriteCastForCFRetain(CastExpr *castE, CallExpr *callE) { |
| 277 | Transaction Trans(Pass.TA); |
| 278 | Pass.TA.replace(callE->getSourceRange(), callE->getArg(0)->getSourceRange()); |
| 279 | rewriteToBridgedCast(castE, OBC_BridgeRetained, Trans); |
| 280 | } |
| 281 | |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 282 | void getBlockMacroRanges(CastExpr *E, SourceRange &Outer, SourceRange &Inner) { |
| 283 | SourceManager &SM = Pass.Ctx.getSourceManager(); |
| 284 | SourceLocation Loc = E->getExprLoc(); |
| 285 | assert(Loc.isMacroID()); |
| 286 | SourceLocation MacroBegin, MacroEnd; |
Benjamin Kramer | 867ea1d | 2014-03-02 13:01:17 +0000 | [diff] [blame] | 287 | std::tie(MacroBegin, MacroEnd) = SM.getImmediateExpansionRange(Loc); |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 288 | SourceRange SubRange = E->getSubExpr()->IgnoreParenImpCasts()->getSourceRange(); |
| 289 | SourceLocation InnerBegin = SM.getImmediateMacroCallerLoc(SubRange.getBegin()); |
| 290 | SourceLocation InnerEnd = SM.getImmediateMacroCallerLoc(SubRange.getEnd()); |
| 291 | |
| 292 | Outer = SourceRange(MacroBegin, MacroEnd); |
| 293 | Inner = SourceRange(InnerBegin, InnerEnd); |
| 294 | } |
| 295 | |
| 296 | void rewriteBlockCopyMacro(CastExpr *E) { |
| 297 | SourceRange OuterRange, InnerRange; |
| 298 | getBlockMacroRanges(E, OuterRange, InnerRange); |
| 299 | |
| 300 | Transaction Trans(Pass.TA); |
| 301 | Pass.TA.replace(OuterRange, InnerRange); |
| 302 | Pass.TA.insert(InnerRange.getBegin(), "["); |
| 303 | Pass.TA.insertAfterToken(InnerRange.getEnd(), " copy]"); |
| 304 | Pass.TA.clearDiagnostic(diag::err_arc_mismatched_cast, |
| 305 | diag::err_arc_cast_requires_bridge, |
| 306 | OuterRange); |
| 307 | } |
| 308 | |
| 309 | void removeBlockReleaseMacro(CastExpr *E) { |
| 310 | SourceRange OuterRange, InnerRange; |
| 311 | getBlockMacroRanges(E, OuterRange, InnerRange); |
| 312 | |
| 313 | Transaction Trans(Pass.TA); |
| 314 | Pass.TA.clearDiagnostic(diag::err_arc_mismatched_cast, |
| 315 | diag::err_arc_cast_requires_bridge, |
| 316 | OuterRange); |
| 317 | if (!hasSideEffects(E, Pass.Ctx)) { |
| 318 | if (tryRemoving(cast<Expr>(StmtMap->getParentIgnoreParenCasts(E)))) |
| 319 | return; |
| 320 | } |
| 321 | Pass.TA.replace(OuterRange, InnerRange); |
| 322 | } |
| 323 | |
| 324 | bool tryRemoving(Expr *E) const { |
| 325 | if (!Removables) { |
| 326 | Removables.reset(new ExprSet); |
| 327 | collectRemovables(Body, *Removables); |
| 328 | } |
| 329 | |
| 330 | if (Removables->count(E)) { |
| 331 | Pass.TA.removeStmt(E); |
| 332 | return true; |
| 333 | } |
| 334 | |
| 335 | return false; |
| 336 | } |
| 337 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 338 | void transformObjCToNonObjCCast(CastExpr *E) { |
Argyrios Kyrtzidis | 6ba7afb | 2013-01-03 03:17:17 +0000 | [diff] [blame] | 339 | SourceLocation CastLoc = E->getExprLoc(); |
| 340 | if (CastLoc.isMacroID()) { |
| 341 | StringRef MacroName = Lexer::getImmediateMacroName(CastLoc, |
| 342 | Pass.Ctx.getSourceManager(), |
| 343 | Pass.Ctx.getLangOpts()); |
| 344 | if (MacroName == "Block_copy") { |
| 345 | rewriteBlockCopyMacro(E); |
| 346 | return; |
| 347 | } |
| 348 | if (MacroName == "Block_release") { |
| 349 | removeBlockReleaseMacro(E); |
| 350 | return; |
| 351 | } |
| 352 | } |
| 353 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 354 | if (isSelf(E->getSubExpr())) |
| 355 | return rewriteToBridgedCast(E, OBC_Bridge); |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 356 | |
| 357 | CallExpr *callE; |
| 358 | if (isPassedToCFRetain(E, callE)) |
| 359 | return rewriteCastForCFRetain(E, callE); |
| 360 | |
| 361 | ObjCMethodFamily family = getFamilyOfMessage(E->getSubExpr()); |
| 362 | if (family == OMF_retain) |
| 363 | return rewriteToBridgedCast(E, OBC_BridgeRetained); |
| 364 | |
| 365 | if (family == OMF_autorelease || family == OMF_release) { |
| 366 | std::string err = "it is not safe to cast to '"; |
Douglas Gregor | c0b0728 | 2011-09-27 22:38:19 +0000 | [diff] [blame] | 367 | err += E->getType().getAsString(Pass.Ctx.getPrintingPolicy()); |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 368 | err += "' the result of '"; |
| 369 | err += family == OMF_autorelease ? "autorelease" : "release"; |
| 370 | err += "' message; a __bridge cast may result in a pointer to a " |
| 371 | "destroyed object and a __bridge_retained may leak the object"; |
| 372 | Pass.TA.reportError(err, E->getLocStart(), |
| 373 | E->getSubExpr()->getSourceRange()); |
| 374 | Stmt *parent = E; |
| 375 | do { |
| 376 | parent = StmtMap->getParentIgnoreParenImpCasts(parent); |
| 377 | } while (parent && isa<ExprWithCleanups>(parent)); |
| 378 | |
| 379 | if (ReturnStmt *retS = dyn_cast_or_null<ReturnStmt>(parent)) { |
| 380 | std::string note = "remove the cast and change return type of function " |
| 381 | "to '"; |
Douglas Gregor | c0b0728 | 2011-09-27 22:38:19 +0000 | [diff] [blame] | 382 | note += E->getSubExpr()->getType().getAsString(Pass.Ctx.getPrintingPolicy()); |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 383 | note += "' to have the object automatically autoreleased"; |
| 384 | Pass.TA.reportNote(note, retS->getLocStart()); |
| 385 | } |
| 386 | } |
| 387 | |
John McCall | fe96e0b | 2011-11-06 09:01:30 +0000 | [diff] [blame] | 388 | Expr *subExpr = E->getSubExpr(); |
| 389 | |
| 390 | // Look through pseudo-object expressions. |
| 391 | if (PseudoObjectExpr *pseudo = dyn_cast<PseudoObjectExpr>(subExpr)) { |
| 392 | subExpr = pseudo->getResultExpr(); |
| 393 | assert(subExpr && "no result for pseudo-object of non-void type?"); |
| 394 | } |
| 395 | |
| 396 | if (ImplicitCastExpr *implCE = dyn_cast<ImplicitCastExpr>(subExpr)) { |
John McCall | 2d637d2 | 2011-09-10 06:18:15 +0000 | [diff] [blame] | 397 | if (implCE->getCastKind() == CK_ARCConsumeObject) |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 398 | return rewriteToBridgedCast(E, OBC_BridgeRetained); |
John McCall | 2d637d2 | 2011-09-10 06:18:15 +0000 | [diff] [blame] | 399 | if (implCE->getCastKind() == CK_ARCReclaimReturnedObject) |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 400 | return rewriteToBridgedCast(E, OBC_Bridge); |
| 401 | } |
Argyrios Kyrtzidis | 41899f3 | 2011-09-14 18:17:09 +0000 | [diff] [blame] | 402 | |
| 403 | bool isConsumed = false; |
| 404 | if (isPassedToCParamWithKnownOwnership(E, isConsumed)) |
| 405 | return rewriteToBridgedCast(E, isConsumed ? OBC_BridgeRetained |
| 406 | : OBC_Bridge); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 407 | } |
| 408 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 409 | static ObjCMethodFamily getFamilyOfMessage(Expr *E) { |
| 410 | E = E->IgnoreParenCasts(); |
| 411 | if (ObjCMessageExpr *ME = dyn_cast<ObjCMessageExpr>(E)) |
| 412 | return ME->getMethodFamily(); |
| 413 | |
| 414 | return OMF_None; |
| 415 | } |
| 416 | |
| 417 | bool isPassedToCFRetain(Expr *E, CallExpr *&callE) const { |
| 418 | if ((callE = dyn_cast_or_null<CallExpr>( |
| 419 | StmtMap->getParentIgnoreParenImpCasts(E)))) |
| 420 | if (FunctionDecl * |
| 421 | FD = dyn_cast_or_null<FunctionDecl>(callE->getCalleeDecl())) |
| 422 | if (FD->getName() == "CFRetain" && FD->getNumParams() == 1 && |
| 423 | FD->getParent()->isTranslationUnit() && |
Rafael Espindola | 3ae0005 | 2013-05-13 00:12:11 +0000 | [diff] [blame] | 424 | FD->isExternallyVisible()) |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 425 | return true; |
| 426 | |
| 427 | return false; |
| 428 | } |
| 429 | |
Argyrios Kyrtzidis | 41899f3 | 2011-09-14 18:17:09 +0000 | [diff] [blame] | 430 | bool isPassedToCParamWithKnownOwnership(Expr *E, bool &isConsumed) const { |
| 431 | if (CallExpr *callE = dyn_cast_or_null<CallExpr>( |
| 432 | StmtMap->getParentIgnoreParenImpCasts(E))) |
| 433 | if (FunctionDecl * |
| 434 | FD = dyn_cast_or_null<FunctionDecl>(callE->getCalleeDecl())) { |
| 435 | unsigned i = 0; |
| 436 | for (unsigned e = callE->getNumArgs(); i != e; ++i) { |
| 437 | Expr *arg = callE->getArg(i); |
| 438 | if (arg == E || arg->IgnoreParenImpCasts() == E) |
| 439 | break; |
| 440 | } |
Argyrios Kyrtzidis | d07fabf | 2013-02-14 17:29:16 +0000 | [diff] [blame] | 441 | if (i < callE->getNumArgs() && i < FD->getNumParams()) { |
Argyrios Kyrtzidis | 41899f3 | 2011-09-14 18:17:09 +0000 | [diff] [blame] | 442 | ParmVarDecl *PD = FD->getParamDecl(i); |
Aaron Ballman | 9ead124 | 2013-12-19 02:39:40 +0000 | [diff] [blame] | 443 | if (PD->hasAttr<CFConsumedAttr>()) { |
Argyrios Kyrtzidis | 41899f3 | 2011-09-14 18:17:09 +0000 | [diff] [blame] | 444 | isConsumed = true; |
| 445 | return true; |
| 446 | } |
| 447 | } |
| 448 | } |
| 449 | |
| 450 | return false; |
| 451 | } |
| 452 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 453 | bool isSelf(Expr *E) const { |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 454 | E = E->IgnoreParenLValueCasts(); |
| 455 | if (DeclRefExpr *DRE = dyn_cast<DeclRefExpr>(E)) |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 456 | if (ImplicitParamDecl *IPD = dyn_cast<ImplicitParamDecl>(DRE->getDecl())) |
| 457 | if (IPD->getIdentifier() == SelfII) |
| 458 | return true; |
| 459 | |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 460 | return false; |
| 461 | } |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 462 | }; |
| 463 | |
| 464 | } // end anonymous namespace |
| 465 | |
| 466 | void trans::rewriteUnbridgedCasts(MigrationPass &pass) { |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 467 | BodyTransform<UnbridgedCastRewriter> trans(pass); |
Argyrios Kyrtzidis | e5b475c | 2011-06-21 20:20:39 +0000 | [diff] [blame] | 468 | trans.TraverseDecl(pass.Ctx.getTranslationUnitDecl()); |
| 469 | } |