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 | f2a27f40d658446fd544f6f4f226b1e1ea6bb1c4 (patch) | |
| tree | e58045dfa2deb96db5438de9daa70abbb4eb5254 /clang/lib/ARCMigrate/Transforms.cpp | |
| parent | 4771159f9fed25993673f4fd321a24a643d6b790 (diff) | |
| download | bcm5719-llvm-f2a27f40d658446fd544f6f4f226b1e1ea6bb1c4.tar.gz bcm5719-llvm-f2a27f40d658446fd544f6f4f226b1e1ea6bb1c4.zip | |
[arcmt] Don't remove retains/releases on a global variable, flag them with errors. rdar://9402555.
llvm-svn: 135213
Diffstat (limited to 'clang/lib/ARCMigrate/Transforms.cpp')
| -rw-r--r-- | clang/lib/ARCMigrate/Transforms.cpp | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/clang/lib/ARCMigrate/Transforms.cpp b/clang/lib/ARCMigrate/Transforms.cpp index 7a439d95e3a..7bd95e54bc2 100644 --- a/clang/lib/ARCMigrate/Transforms.cpp +++ b/clang/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> { |
