forked from OSchip/llvm-project
Remove Decl::getCompoundBody().
This has 2 (slight) advantages: -Make explicit at getBody()'s callsite that we expect/handle only CompoundStmt and not CXXTryStmt. -Better tracking of Decl::getBody()'s callsites. llvm-svn: 107771
This commit is contained in:
parent
6fbc8fa53e
commit
568bc84562
|
@ -492,9 +492,6 @@ public:
|
|||
/// code, such as a function or method definition.
|
||||
virtual bool hasBody() const { return getBody() != 0; }
|
||||
|
||||
/// getCompoundBody - Returns getBody(), dyn_casted to a CompoundStmt.
|
||||
CompoundStmt* getCompoundBody() const;
|
||||
|
||||
/// getBodyRBrace - Gets the right brace of the body, if a body exists.
|
||||
/// This works whether the body is a CompoundStmt or a CXXTryStmt.
|
||||
SourceLocation getBodyRBrace() const;
|
||||
|
|
|
@ -447,10 +447,6 @@ DeclContext *Decl::castToDeclContext(const Decl *D) {
|
|||
}
|
||||
}
|
||||
|
||||
CompoundStmt* Decl::getCompoundBody() const {
|
||||
return dyn_cast_or_null<CompoundStmt>(getBody());
|
||||
}
|
||||
|
||||
SourceLocation Decl::getBodyRBrace() const {
|
||||
// Special handling of FunctionDecl to avoid de-serializing the body from PCH.
|
||||
// FunctionDecl stores EndRangeLoc for this purpose.
|
||||
|
|
|
@ -925,7 +925,7 @@ public:
|
|||
// statement (if it doesn't already exist).
|
||||
// FIXME: Should handle CXXTryStmt if analyser starts supporting C++.
|
||||
if (const CompoundStmt *CS =
|
||||
PDB.getCodeDecl().getCompoundBody())
|
||||
dyn_cast_or_null<CompoundStmt>(PDB.getCodeDecl().getBody()))
|
||||
if (!CS->body_empty()) {
|
||||
SourceLocation Loc = (*CS->body_begin())->getLocStart();
|
||||
rawAddEdge(PathDiagnosticLocation(Loc, PDB.getSourceManager()));
|
||||
|
|
|
@ -5651,7 +5651,7 @@ void RewriteObjC::HandleDeclInMainFile(Decl *D) {
|
|||
RewriteBlocksInFunctionProtoType(FD->getType(), FD);
|
||||
|
||||
// FIXME: If this should support Obj-C++, support CXXTryStmt
|
||||
if (CompoundStmt *Body = FD->getCompoundBody()) {
|
||||
if (CompoundStmt *Body = dyn_cast_or_null<CompoundStmt>(FD->getBody())) {
|
||||
CurFunctionDef = FD;
|
||||
CurFunctionDeclToDeclareForBlock = FD;
|
||||
CollectPropertySetters(Body);
|
||||
|
|
Loading…
Reference in New Issue