1524b3c18SFangrui Song //===-- TransEmptyStatementsAndDealloc.cpp - Transformations to ARC mode --===//
2e5acb84eSArgyrios Kyrtzidis //
32946cd70SChandler Carruth // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
42946cd70SChandler Carruth // See https://llvm.org/LICENSE.txt for license information.
52946cd70SChandler Carruth // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6e5acb84eSArgyrios Kyrtzidis //
7e5acb84eSArgyrios Kyrtzidis //===----------------------------------------------------------------------===//
8e5acb84eSArgyrios Kyrtzidis //
9e5acb84eSArgyrios Kyrtzidis // removeEmptyStatementsAndDealloc:
10e5acb84eSArgyrios Kyrtzidis //
11e5acb84eSArgyrios Kyrtzidis // Removes empty statements that are leftovers from previous transformations.
12e5acb84eSArgyrios Kyrtzidis // e.g for
13e5acb84eSArgyrios Kyrtzidis //
14e5acb84eSArgyrios Kyrtzidis //  [x retain];
15e5acb84eSArgyrios Kyrtzidis //
16e5acb84eSArgyrios Kyrtzidis // removeRetainReleaseDealloc will leave an empty ";" that removeEmptyStatements
17e5acb84eSArgyrios Kyrtzidis // will remove.
18e5acb84eSArgyrios Kyrtzidis //
19e5acb84eSArgyrios Kyrtzidis //===----------------------------------------------------------------------===//
20e5acb84eSArgyrios Kyrtzidis 
21e5acb84eSArgyrios Kyrtzidis #include "Transforms.h"
22e5acb84eSArgyrios Kyrtzidis #include "Internals.h"
234ab984e7SBenjamin Kramer #include "clang/AST/ASTContext.h"
24e5acb84eSArgyrios Kyrtzidis #include "clang/AST/StmtVisitor.h"
25cbbc0141SArgyrios Kyrtzidis #include "clang/Basic/SourceManager.h"
26e5acb84eSArgyrios Kyrtzidis 
27e5acb84eSArgyrios Kyrtzidis using namespace clang;
28e5acb84eSArgyrios Kyrtzidis using namespace arcmt;
29e5acb84eSArgyrios Kyrtzidis using namespace trans;
30e5acb84eSArgyrios Kyrtzidis 
isEmptyARCMTMacroStatement(NullStmt * S,std::vector<SourceLocation> & MacroLocs,ASTContext & Ctx)31cbbc0141SArgyrios Kyrtzidis static bool isEmptyARCMTMacroStatement(NullStmt *S,
32cbbc0141SArgyrios Kyrtzidis                                        std::vector<SourceLocation> &MacroLocs,
33cbbc0141SArgyrios Kyrtzidis                                        ASTContext &Ctx) {
34cbbc0141SArgyrios Kyrtzidis   if (!S->hasLeadingEmptyMacro())
35cbbc0141SArgyrios Kyrtzidis     return false;
36cbbc0141SArgyrios Kyrtzidis 
37cbbc0141SArgyrios Kyrtzidis   SourceLocation SemiLoc = S->getSemiLoc();
38cbbc0141SArgyrios Kyrtzidis   if (SemiLoc.isInvalid() || SemiLoc.isMacroID())
39cbbc0141SArgyrios Kyrtzidis     return false;
40cbbc0141SArgyrios Kyrtzidis 
41cbbc0141SArgyrios Kyrtzidis   if (MacroLocs.empty())
42cbbc0141SArgyrios Kyrtzidis     return false;
43cbbc0141SArgyrios Kyrtzidis 
44cbbc0141SArgyrios Kyrtzidis   SourceManager &SM = Ctx.getSourceManager();
457264a474SFangrui Song   std::vector<SourceLocation>::iterator I = llvm::upper_bound(
467264a474SFangrui Song       MacroLocs, SemiLoc, BeforeThanCompare<SourceLocation>(SM));
47cbbc0141SArgyrios Kyrtzidis   --I;
48cbbc0141SArgyrios Kyrtzidis   SourceLocation
49e6e67deeSArgyrios Kyrtzidis       AfterMacroLoc = I->getLocWithOffset(getARCMTMacroName().size());
50cbbc0141SArgyrios Kyrtzidis   assert(AfterMacroLoc.isFileID());
51cbbc0141SArgyrios Kyrtzidis 
52cbbc0141SArgyrios Kyrtzidis   if (AfterMacroLoc == SemiLoc)
53cbbc0141SArgyrios Kyrtzidis     return true;
54cbbc0141SArgyrios Kyrtzidis 
55*21401a72SSimon Tatham   SourceLocation::IntTy RelOffs = 0;
56cbbc0141SArgyrios Kyrtzidis   if (!SM.isInSameSLocAddrSpace(AfterMacroLoc, SemiLoc, &RelOffs))
57cbbc0141SArgyrios Kyrtzidis     return false;
58cbbc0141SArgyrios Kyrtzidis   if (RelOffs < 0)
59cbbc0141SArgyrios Kyrtzidis     return false;
60cbbc0141SArgyrios Kyrtzidis 
61cbbc0141SArgyrios Kyrtzidis   // We make the reasonable assumption that a semicolon after 100 characters
62cbbc0141SArgyrios Kyrtzidis   // means that it is not the next token after our macro. If this assumption
63cbbc0141SArgyrios Kyrtzidis   // fails it is not critical, we will just fail to clear out, e.g., an empty
64cbbc0141SArgyrios Kyrtzidis   // 'if'.
65cbbc0141SArgyrios Kyrtzidis   if (RelOffs - getARCMTMacroName().size() > 100)
66cbbc0141SArgyrios Kyrtzidis     return false;
67cbbc0141SArgyrios Kyrtzidis 
68cbbc0141SArgyrios Kyrtzidis   SourceLocation AfterMacroSemiLoc = findSemiAfterLocation(AfterMacroLoc, Ctx);
69cbbc0141SArgyrios Kyrtzidis   return AfterMacroSemiLoc == SemiLoc;
70cbbc0141SArgyrios Kyrtzidis }
71cbbc0141SArgyrios Kyrtzidis 
72e5acb84eSArgyrios Kyrtzidis namespace {
73e5acb84eSArgyrios Kyrtzidis 
749fc8faf9SAdrian Prantl /// Returns true if the statement became empty due to previous
75e5acb84eSArgyrios Kyrtzidis /// transformations.
76e5acb84eSArgyrios Kyrtzidis class EmptyChecker : public StmtVisitor<EmptyChecker, bool> {
77e5acb84eSArgyrios Kyrtzidis   ASTContext &Ctx;
78cbbc0141SArgyrios Kyrtzidis   std::vector<SourceLocation> &MacroLocs;
79e5acb84eSArgyrios Kyrtzidis 
80e5acb84eSArgyrios Kyrtzidis public:
EmptyChecker(ASTContext & ctx,std::vector<SourceLocation> & macroLocs)81cbbc0141SArgyrios Kyrtzidis   EmptyChecker(ASTContext &ctx, std::vector<SourceLocation> &macroLocs)
82e5acb84eSArgyrios Kyrtzidis     : Ctx(ctx), MacroLocs(macroLocs) { }
83e5acb84eSArgyrios Kyrtzidis 
VisitNullStmt(NullStmt * S)84e5acb84eSArgyrios Kyrtzidis   bool VisitNullStmt(NullStmt *S) {
85cbbc0141SArgyrios Kyrtzidis     return isEmptyARCMTMacroStatement(S, MacroLocs, Ctx);
86e5acb84eSArgyrios Kyrtzidis   }
VisitCompoundStmt(CompoundStmt * S)87e5acb84eSArgyrios Kyrtzidis   bool VisitCompoundStmt(CompoundStmt *S) {
88e5acb84eSArgyrios Kyrtzidis     if (S->body_empty())
89e5acb84eSArgyrios Kyrtzidis       return false; // was already empty, not because of transformations.
90c7e4e219SAaron Ballman     for (auto *I : S->body())
91c7e4e219SAaron Ballman       if (!Visit(I))
92e5acb84eSArgyrios Kyrtzidis         return false;
93e5acb84eSArgyrios Kyrtzidis     return true;
94e5acb84eSArgyrios Kyrtzidis   }
VisitIfStmt(IfStmt * S)95e5acb84eSArgyrios Kyrtzidis   bool VisitIfStmt(IfStmt *S) {
96e5acb84eSArgyrios Kyrtzidis     if (S->getConditionVariable())
97e5acb84eSArgyrios Kyrtzidis       return false;
98e5acb84eSArgyrios Kyrtzidis     Expr *condE = S->getCond();
99e5acb84eSArgyrios Kyrtzidis     if (!condE)
100e5acb84eSArgyrios Kyrtzidis       return false;
101e5acb84eSArgyrios Kyrtzidis     if (hasSideEffects(condE, Ctx))
102e5acb84eSArgyrios Kyrtzidis       return false;
103e5acb84eSArgyrios Kyrtzidis     if (!S->getThen() || !Visit(S->getThen()))
104e5acb84eSArgyrios Kyrtzidis       return false;
105ad98885bSAlexander Kornienko     return !S->getElse() || Visit(S->getElse());
106e5acb84eSArgyrios Kyrtzidis   }
VisitWhileStmt(WhileStmt * S)107e5acb84eSArgyrios Kyrtzidis   bool VisitWhileStmt(WhileStmt *S) {
108e5acb84eSArgyrios Kyrtzidis     if (S->getConditionVariable())
109e5acb84eSArgyrios Kyrtzidis       return false;
110e5acb84eSArgyrios Kyrtzidis     Expr *condE = S->getCond();
111e5acb84eSArgyrios Kyrtzidis     if (!condE)
112e5acb84eSArgyrios Kyrtzidis       return false;
113e5acb84eSArgyrios Kyrtzidis     if (hasSideEffects(condE, Ctx))
114e5acb84eSArgyrios Kyrtzidis       return false;
115e5acb84eSArgyrios Kyrtzidis     if (!S->getBody())
116e5acb84eSArgyrios Kyrtzidis       return false;
117e5acb84eSArgyrios Kyrtzidis     return Visit(S->getBody());
118e5acb84eSArgyrios Kyrtzidis   }
VisitDoStmt(DoStmt * S)119e5acb84eSArgyrios Kyrtzidis   bool VisitDoStmt(DoStmt *S) {
120e5acb84eSArgyrios Kyrtzidis     Expr *condE = S->getCond();
121e5acb84eSArgyrios Kyrtzidis     if (!condE)
122e5acb84eSArgyrios Kyrtzidis       return false;
123e5acb84eSArgyrios Kyrtzidis     if (hasSideEffects(condE, Ctx))
124e5acb84eSArgyrios Kyrtzidis       return false;
125e5acb84eSArgyrios Kyrtzidis     if (!S->getBody())
126e5acb84eSArgyrios Kyrtzidis       return false;
127e5acb84eSArgyrios Kyrtzidis     return Visit(S->getBody());
128e5acb84eSArgyrios Kyrtzidis   }
VisitObjCForCollectionStmt(ObjCForCollectionStmt * S)129e5acb84eSArgyrios Kyrtzidis   bool VisitObjCForCollectionStmt(ObjCForCollectionStmt *S) {
130e5acb84eSArgyrios Kyrtzidis     Expr *Exp = S->getCollection();
131e5acb84eSArgyrios Kyrtzidis     if (!Exp)
132e5acb84eSArgyrios Kyrtzidis       return false;
133e5acb84eSArgyrios Kyrtzidis     if (hasSideEffects(Exp, Ctx))
134e5acb84eSArgyrios Kyrtzidis       return false;
135e5acb84eSArgyrios Kyrtzidis     if (!S->getBody())
136e5acb84eSArgyrios Kyrtzidis       return false;
137e5acb84eSArgyrios Kyrtzidis     return Visit(S->getBody());
138e5acb84eSArgyrios Kyrtzidis   }
VisitObjCAutoreleasePoolStmt(ObjCAutoreleasePoolStmt * S)139e5acb84eSArgyrios Kyrtzidis   bool VisitObjCAutoreleasePoolStmt(ObjCAutoreleasePoolStmt *S) {
140e5acb84eSArgyrios Kyrtzidis     if (!S->getSubStmt())
141e5acb84eSArgyrios Kyrtzidis       return false;
142e5acb84eSArgyrios Kyrtzidis     return Visit(S->getSubStmt());
143e5acb84eSArgyrios Kyrtzidis   }
144e5acb84eSArgyrios Kyrtzidis };
145e5acb84eSArgyrios Kyrtzidis 
146e5acb84eSArgyrios Kyrtzidis class EmptyStatementsRemover :
147e5acb84eSArgyrios Kyrtzidis                             public RecursiveASTVisitor<EmptyStatementsRemover> {
148e5acb84eSArgyrios Kyrtzidis   MigrationPass &Pass;
149e5acb84eSArgyrios Kyrtzidis 
150e5acb84eSArgyrios Kyrtzidis public:
EmptyStatementsRemover(MigrationPass & pass)151cbbc0141SArgyrios Kyrtzidis   EmptyStatementsRemover(MigrationPass &pass) : Pass(pass) { }
152e5acb84eSArgyrios Kyrtzidis 
TraverseStmtExpr(StmtExpr * E)153e5acb84eSArgyrios Kyrtzidis   bool TraverseStmtExpr(StmtExpr *E) {
154e5acb84eSArgyrios Kyrtzidis     CompoundStmt *S = E->getSubStmt();
155e5acb84eSArgyrios Kyrtzidis     for (CompoundStmt::body_iterator
156e5acb84eSArgyrios Kyrtzidis            I = S->body_begin(), E = S->body_end(); I != E; ++I) {
157e5acb84eSArgyrios Kyrtzidis       if (I != E - 1)
158e5acb84eSArgyrios Kyrtzidis         check(*I);
159e5acb84eSArgyrios Kyrtzidis       TraverseStmt(*I);
160e5acb84eSArgyrios Kyrtzidis     }
161e5acb84eSArgyrios Kyrtzidis     return true;
162e5acb84eSArgyrios Kyrtzidis   }
163e5acb84eSArgyrios Kyrtzidis 
VisitCompoundStmt(CompoundStmt * S)164e5acb84eSArgyrios Kyrtzidis   bool VisitCompoundStmt(CompoundStmt *S) {
165c7e4e219SAaron Ballman     for (auto *I : S->body())
166c7e4e219SAaron Ballman       check(I);
167e5acb84eSArgyrios Kyrtzidis     return true;
168e5acb84eSArgyrios Kyrtzidis   }
169e5acb84eSArgyrios Kyrtzidis 
getContext()170e5acb84eSArgyrios Kyrtzidis   ASTContext &getContext() { return Pass.Ctx; }
171e5acb84eSArgyrios Kyrtzidis 
172e5acb84eSArgyrios Kyrtzidis private:
check(Stmt * S)173e5acb84eSArgyrios Kyrtzidis   void check(Stmt *S) {
174e5acb84eSArgyrios Kyrtzidis     if (!S) return;
175cbbc0141SArgyrios Kyrtzidis     if (EmptyChecker(Pass.Ctx, Pass.ARCMTMacroLocs).Visit(S)) {
176e5acb84eSArgyrios Kyrtzidis       Transaction Trans(Pass.TA);
177e5acb84eSArgyrios Kyrtzidis       Pass.TA.removeStmt(S);
178e5acb84eSArgyrios Kyrtzidis     }
179e5acb84eSArgyrios Kyrtzidis   }
180e5acb84eSArgyrios Kyrtzidis };
181e5acb84eSArgyrios Kyrtzidis 
182e5acb84eSArgyrios Kyrtzidis } // anonymous namespace
183e5acb84eSArgyrios Kyrtzidis 
isBodyEmpty(CompoundStmt * body,ASTContext & Ctx,std::vector<SourceLocation> & MacroLocs)184cbbc0141SArgyrios Kyrtzidis static bool isBodyEmpty(CompoundStmt *body, ASTContext &Ctx,
185cbbc0141SArgyrios Kyrtzidis                         std::vector<SourceLocation> &MacroLocs) {
186c7e4e219SAaron Ballman   for (auto *I : body->body())
187c7e4e219SAaron Ballman     if (!EmptyChecker(Ctx, MacroLocs).Visit(I))
188e5acb84eSArgyrios Kyrtzidis       return false;
189e5acb84eSArgyrios Kyrtzidis 
190e5acb84eSArgyrios Kyrtzidis   return true;
191e5acb84eSArgyrios Kyrtzidis }
192e5acb84eSArgyrios Kyrtzidis 
cleanupDeallocOrFinalize(MigrationPass & pass)193d2b91123SArgyrios Kyrtzidis static void cleanupDeallocOrFinalize(MigrationPass &pass) {
194e5acb84eSArgyrios Kyrtzidis   ASTContext &Ctx = pass.Ctx;
195e5acb84eSArgyrios Kyrtzidis   TransformActions &TA = pass.TA;
196e5acb84eSArgyrios Kyrtzidis   DeclContext *DC = Ctx.getTranslationUnitDecl();
197d2b91123SArgyrios Kyrtzidis   Selector FinalizeSel =
198d2b91123SArgyrios Kyrtzidis       Ctx.Selectors.getNullarySelector(&pass.Ctx.Idents.get("finalize"));
199e5acb84eSArgyrios Kyrtzidis 
200e5acb84eSArgyrios Kyrtzidis   typedef DeclContext::specific_decl_iterator<ObjCImplementationDecl>
201e5acb84eSArgyrios Kyrtzidis     impl_iterator;
202e5acb84eSArgyrios Kyrtzidis   for (impl_iterator I = impl_iterator(DC->decls_begin()),
203e5acb84eSArgyrios Kyrtzidis                      E = impl_iterator(DC->decls_end()); I != E; ++I) {
2048ae12039SCraig Topper     ObjCMethodDecl *DeallocM = nullptr;
2058ae12039SCraig Topper     ObjCMethodDecl *FinalizeM = nullptr;
206f26acce6SAaron Ballman     for (auto *MD : I->instance_methods()) {
207d2b91123SArgyrios Kyrtzidis       if (!MD->hasBody())
208d2b91123SArgyrios Kyrtzidis         continue;
209d2b91123SArgyrios Kyrtzidis 
210e5acb84eSArgyrios Kyrtzidis       if (MD->getMethodFamily() == OMF_dealloc) {
211d2b91123SArgyrios Kyrtzidis         DeallocM = MD;
212d2b91123SArgyrios Kyrtzidis       } else if (MD->isInstanceMethod() && MD->getSelector() == FinalizeSel) {
213d2b91123SArgyrios Kyrtzidis         FinalizeM = MD;
214e5acb84eSArgyrios Kyrtzidis       }
215d2b91123SArgyrios Kyrtzidis     }
216d2b91123SArgyrios Kyrtzidis 
217d2b91123SArgyrios Kyrtzidis     if (DeallocM) {
218d2b91123SArgyrios Kyrtzidis       if (isBodyEmpty(DeallocM->getCompoundBody(), Ctx, pass.ARCMTMacroLocs)) {
219d2b91123SArgyrios Kyrtzidis         Transaction Trans(TA);
220d2b91123SArgyrios Kyrtzidis         TA.remove(DeallocM->getSourceRange());
221d2b91123SArgyrios Kyrtzidis       }
222d2b91123SArgyrios Kyrtzidis 
223d2b91123SArgyrios Kyrtzidis       if (FinalizeM) {
224d2b91123SArgyrios Kyrtzidis         Transaction Trans(TA);
225d2b91123SArgyrios Kyrtzidis         TA.remove(FinalizeM->getSourceRange());
226d2b91123SArgyrios Kyrtzidis       }
227d2b91123SArgyrios Kyrtzidis 
228d2b91123SArgyrios Kyrtzidis     } else if (FinalizeM) {
229d2b91123SArgyrios Kyrtzidis       if (isBodyEmpty(FinalizeM->getCompoundBody(), Ctx, pass.ARCMTMacroLocs)) {
230d2b91123SArgyrios Kyrtzidis         Transaction Trans(TA);
231d2b91123SArgyrios Kyrtzidis         TA.remove(FinalizeM->getSourceRange());
232d2b91123SArgyrios Kyrtzidis       } else {
233d2b91123SArgyrios Kyrtzidis         Transaction Trans(TA);
234d2b91123SArgyrios Kyrtzidis         TA.replaceText(FinalizeM->getSelectorStartLoc(), "finalize", "dealloc");
235e5acb84eSArgyrios Kyrtzidis       }
236e5acb84eSArgyrios Kyrtzidis     }
237e5acb84eSArgyrios Kyrtzidis   }
238e5acb84eSArgyrios Kyrtzidis }
239e5acb84eSArgyrios Kyrtzidis 
removeEmptyStatementsAndDeallocFinalize(MigrationPass & pass)240d2b91123SArgyrios Kyrtzidis void trans::removeEmptyStatementsAndDeallocFinalize(MigrationPass &pass) {
241cbbc0141SArgyrios Kyrtzidis   EmptyStatementsRemover(pass).TraverseDecl(pass.Ctx.getTranslationUnitDecl());
242e5acb84eSArgyrios Kyrtzidis 
243d2b91123SArgyrios Kyrtzidis   cleanupDeallocOrFinalize(pass);
244e5acb84eSArgyrios Kyrtzidis 
245e5acb84eSArgyrios Kyrtzidis   for (unsigned i = 0, e = pass.ARCMTMacroLocs.size(); i != e; ++i) {
246e5acb84eSArgyrios Kyrtzidis     Transaction Trans(pass.TA);
247e5acb84eSArgyrios Kyrtzidis     pass.TA.remove(pass.ARCMTMacroLocs[i]);
248e5acb84eSArgyrios Kyrtzidis   }
249e5acb84eSArgyrios Kyrtzidis }
250