For the -dealloc checker, check the LangOptions to determine whether or not the code is compiled with GC.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@53098 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/Driver/AnalysisConsumer.cpp b/Driver/AnalysisConsumer.cpp
index 1dc2fac..44d96a0 100644
--- a/Driver/AnalysisConsumer.cpp
+++ b/Driver/AnalysisConsumer.cpp
@@ -385,7 +385,9 @@
static void ActionCheckObjCDealloc(AnalysisManager& mgr) {
BugReporter BR(mgr);
- CheckObjCDealloc(cast<ObjCImplementationDecl>(mgr.getCodeDecl()), BR);
+
+ CheckObjCDealloc(cast<ObjCImplementationDecl>(mgr.getCodeDecl()),
+ mgr.getLangOptions(), BR);
}
//===----------------------------------------------------------------------===//
@@ -439,7 +441,8 @@
}
// Checks we always perform:
- C->addObjCImplementationAction(&ActionCheckObjCDealloc);
+ if (lopts.getGCMode() != LangOptions::GCOnly)
+ C->addObjCImplementationAction(&ActionCheckObjCDealloc);
return C.take();
}
diff --git a/include/clang/Analysis/LocalCheckers.h b/include/clang/Analysis/LocalCheckers.h
index 08959d3..acf789d 100644
--- a/include/clang/Analysis/LocalCheckers.h
+++ b/include/clang/Analysis/LocalCheckers.h
@@ -29,6 +29,7 @@
class LiveVariables;
class BugReporter;
class ObjCImplementationDecl;
+class LangOptions;
void CheckDeadStores(LiveVariables& L, BugReporter& BR);
@@ -40,7 +41,8 @@
bool StandardWarnings,
const LangOptions& lopts);
-void CheckObjCDealloc(ObjCImplementationDecl* D, BugReporter& BR);
+void CheckObjCDealloc(ObjCImplementationDecl* D, const LangOptions& L,
+ BugReporter& BR);
} // end namespace clang
diff --git a/lib/Analysis/CheckObjCDealloc.cpp b/lib/Analysis/CheckObjCDealloc.cpp
index 9a40fa9..38736df 100644
--- a/lib/Analysis/CheckObjCDealloc.cpp
+++ b/lib/Analysis/CheckObjCDealloc.cpp
@@ -18,6 +18,7 @@
#include "clang/AST/ExprObjC.h"
#include "clang/AST/Expr.h"
#include "clang/AST/DeclObjC.h"
+#include "clang/Basic/LangOptions.h"
#include <sstream>
using namespace clang;
@@ -40,8 +41,11 @@
return false;
}
-void clang::CheckObjCDealloc(ObjCImplementationDecl* D, BugReporter& BR) {
+void clang::CheckObjCDealloc(ObjCImplementationDecl* D,
+ const LangOptions& LOpts, BugReporter& BR) {
+ assert (LOpts.getGCMode() != LangOptions::GCOnly);
+
ASTContext& Ctx = BR.getContext();
// Determine if the class subclasses NSObject.
@@ -74,7 +78,10 @@
if (!MD) { // No dealloc found.
// FIXME: This code should be reduced to three lines if possible (Refactor).
- SimpleBugType BT("missing -dealloc");
+ SimpleBugType BT(LOpts.getGCMode() == LangOptions::NonGC
+ ? "missing -dealloc"
+ : "missing -dealloc (Hybrid MM, non-GC)");
+
DiagCollector C(BT);
std::ostringstream os;
@@ -97,12 +104,15 @@
if (MD->getBody() && !scan_dealloc(MD->getBody(), S)) {
// FIXME: This code should be reduced to three lines if possible (Refactor).
- SimpleBugType BT("missing [super dealloc]");
+ SimpleBugType BT(LOpts.getGCMode() == LangOptions::NonGC
+ ? "missing [super dealloc]"
+ : "missing [super dealloc] (Hybrid MM, non-GC)");
+
DiagCollector C(BT);
std::ostringstream os;
os << "The 'dealloc' instance method in Objective-C class '" << D->getName()
- << "' does not send a 'dealloc' message to it super class"
+ << "' does not send a 'dealloc' message to its super class"
" (missing [super dealloc])";
Diagnostic& Diag = BR.getDiagnostic();