Benjamin Kramer | d81108f | 2012-11-14 15:08:31 +0000 | [diff] [blame] | 1 | //===--- TransAPIUses.cpp - Transformations to ARC mode -------------------===// |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +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 | // checkAPIUses: |
| 11 | // |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 12 | // Emits error/fix with some API uses that are obsolete or not safe in ARC mode: |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 13 | // |
| 14 | // - NSInvocation's [get/set]ReturnValue and [get/set]Argument are only safe |
| 15 | // with __unsafe_unretained objects. |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 16 | // - Calling -zone gets replaced with 'nil'. |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 17 | // |
| 18 | //===----------------------------------------------------------------------===// |
| 19 | |
| 20 | #include "Transforms.h" |
| 21 | #include "Internals.h" |
Benjamin Kramer | 4ab984e | 2012-07-04 20:19:54 +0000 | [diff] [blame] | 22 | #include "clang/AST/ASTContext.h" |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 23 | #include "clang/Sema/SemaDiagnostic.h" |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 24 | |
| 25 | using namespace clang; |
| 26 | using namespace arcmt; |
| 27 | using namespace trans; |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 28 | |
| 29 | namespace { |
| 30 | |
| 31 | class APIChecker : public RecursiveASTVisitor<APIChecker> { |
| 32 | MigrationPass &Pass; |
Argyrios Kyrtzidis | 91c62bf | 2011-07-18 07:44:50 +0000 | [diff] [blame] | 33 | |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 34 | Selector getReturnValueSel, setReturnValueSel; |
| 35 | Selector getArgumentSel, setArgumentSel; |
| 36 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 37 | Selector zoneSel; |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 38 | public: |
| 39 | APIChecker(MigrationPass &pass) : Pass(pass) { |
| 40 | SelectorTable &sels = Pass.Ctx.Selectors; |
| 41 | IdentifierTable &ids = Pass.Ctx.Idents; |
| 42 | getReturnValueSel = sels.getUnarySelector(&ids.get("getReturnValue")); |
| 43 | setReturnValueSel = sels.getUnarySelector(&ids.get("setReturnValue")); |
| 44 | |
| 45 | IdentifierInfo *selIds[2]; |
| 46 | selIds[0] = &ids.get("getArgument"); |
| 47 | selIds[1] = &ids.get("atIndex"); |
| 48 | getArgumentSel = sels.getSelector(2, selIds); |
| 49 | selIds[0] = &ids.get("setArgument"); |
| 50 | setArgumentSel = sels.getSelector(2, selIds); |
Argyrios Kyrtzidis | 91c62bf | 2011-07-18 07:44:50 +0000 | [diff] [blame] | 51 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 52 | zoneSel = sels.getNullarySelector(&ids.get("zone")); |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 53 | } |
| 54 | |
| 55 | bool VisitObjCMessageExpr(ObjCMessageExpr *E) { |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 56 | // NSInvocation. |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 57 | if (E->isInstanceMessage() && |
| 58 | E->getReceiverInterface() && |
| 59 | E->getReceiverInterface()->getName() == "NSInvocation") { |
| 60 | StringRef selName; |
| 61 | if (E->getSelector() == getReturnValueSel) |
| 62 | selName = "getReturnValue"; |
| 63 | else if (E->getSelector() == setReturnValueSel) |
| 64 | selName = "setReturnValue"; |
| 65 | else if (E->getSelector() == getArgumentSel) |
| 66 | selName = "getArgument"; |
| 67 | else if (E->getSelector() == setArgumentSel) |
| 68 | selName = "setArgument"; |
Alp Toker | 42aa212 | 2014-01-26 05:07:32 +0000 | [diff] [blame] | 69 | else |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 70 | return true; |
| 71 | |
| 72 | Expr *parm = E->getArg(0)->IgnoreParenCasts(); |
| 73 | QualType pointee = parm->getType()->getPointeeType(); |
| 74 | if (pointee.isNull()) |
| 75 | return true; |
| 76 | |
Alp Toker | 42aa212 | 2014-01-26 05:07:32 +0000 | [diff] [blame] | 77 | if (pointee.getObjCLifetime() > Qualifiers::OCL_ExplicitNone) |
Stephen Kelly | f2ceec4 | 2018-08-09 21:08:08 +0000 | [diff] [blame] | 78 | Pass.TA.report(parm->getBeginLoc(), |
Alp Toker | 42aa212 | 2014-01-26 05:07:32 +0000 | [diff] [blame] | 79 | diag::err_arcmt_nsinvocation_ownership, |
| 80 | parm->getSourceRange()) |
| 81 | << selName; |
| 82 | |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 83 | return true; |
| 84 | } |
| 85 | |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 86 | // -zone. |
| 87 | if (E->isInstanceMessage() && |
| 88 | E->getInstanceReceiver() && |
| 89 | E->getSelector() == zoneSel && |
| 90 | Pass.TA.hasDiagnostic(diag::err_unavailable, |
| 91 | diag::err_unavailable_message, |
Argyrios Kyrtzidis | bcf2bdc | 2013-05-01 00:24:09 +0000 | [diff] [blame] | 92 | E->getSelectorLoc(0))) { |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 93 | // Calling -zone is meaningless in ARC, change it to nil. |
| 94 | Transaction Trans(Pass.TA); |
| 95 | Pass.TA.clearDiagnostic(diag::err_unavailable, |
| 96 | diag::err_unavailable_message, |
Argyrios Kyrtzidis | bcf2bdc | 2013-05-01 00:24:09 +0000 | [diff] [blame] | 97 | E->getSelectorLoc(0)); |
Richard Smith | 20e883e | 2015-04-29 23:20:19 +0000 | [diff] [blame] | 98 | Pass.TA.replace(E->getSourceRange(), getNilString(Pass)); |
Argyrios Kyrtzidis | 9390747 | 2011-07-27 05:28:18 +0000 | [diff] [blame] | 99 | } |
Argyrios Kyrtzidis | 73a0d32 | 2011-07-18 07:44:45 +0000 | [diff] [blame] | 100 | return true; |
| 101 | } |
| 102 | }; |
| 103 | |
| 104 | } // anonymous namespace |
| 105 | |
| 106 | void trans::checkAPIUses(MigrationPass &pass) { |
| 107 | APIChecker(pass).TraverseDecl(pass.Ctx.getTranslationUnitDecl()); |
| 108 | } |