diff options
| author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-07-14 23:32:04 +0000 |
|---|---|---|
| committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-07-14 23:32:04 +0000 |
| commit | 2c18ca0575b60082f2a9f4563b4071071960d37c (patch) | |
| tree | cc646d6f0227a762ba6c84fbfd7060b8068f3780 /lib/ARCMigrate/Transforms.cpp | |
| parent | 12ce0a085f89f07c76bf034aa6b838ef50542241 (diff) | |
[arcmt] Don't remove retains/releases on a global variable, flag them with errors. rdar://9402555.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@135213 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/ARCMigrate/Transforms.cpp')
| -rw-r--r-- | lib/ARCMigrate/Transforms.cpp | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/ARCMigrate/Transforms.cpp b/lib/ARCMigrate/Transforms.cpp index 7a439d95e3..7bd95e54bc 100644 --- a/lib/ARCMigrate/Transforms.cpp +++ b/lib/ARCMigrate/Transforms.cpp @@ -152,6 +152,17 @@ bool trans::hasSideEffects(Expr *E, ASTContext &Ctx) { return true; } +bool trans::isGlobalVar(Expr *E) { + E = E->IgnoreParenCasts(); + if (DeclRefExpr *DRE = dyn_cast<DeclRefExpr>(E)) + return DRE->getDecl()->getDeclContext()->isFileContext(); + if (ConditionalOperator *condOp = dyn_cast<ConditionalOperator>(E)) + return isGlobalVar(condOp->getTrueExpr()) && + isGlobalVar(condOp->getFalseExpr()); + + return false; +} + namespace { class ReferenceClear : public RecursiveASTVisitor<ReferenceClear> { |