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> ¯oLocs)
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