maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

make CaseStmt::getSourceRange() iterative for deeply 

nested cases instead of recursive.

llvm-svn: 66013
This commit is contained in:
Chris Lattner 2009-03-04 04:26:45 +00:00
parent 3efa41a478
commit e46f6e7d9c
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
clang/include/clang/AST/Stmt.h
View File

@ -422,7 +422,6 @@ public:
void setNextSwitchCase(SwitchCase *SC) { NextSwitchCase = SC; } void setNextSwitchCase(SwitchCase *SC) { NextSwitchCase = SC; }
virtual Stmt* v_getSubStmt() = 0;
Stmt *getSubStmt() { return v_getSubStmt(); } Stmt *getSubStmt() { return v_getSubStmt(); }
virtual SourceRange getSourceRange() const { return SourceRange(); } virtual SourceRange getSourceRange() const { return SourceRange(); }
@ -432,6 +431,8 @@ public:
T->getStmtClass() == DefaultStmtClass; T->getStmtClass() == DefaultStmtClass;
} }
static bool classof(const SwitchCase *) { return true; } static bool classof(const SwitchCase *) { return true; }
protected:
virtual Stmt* v_getSubStmt() = 0;
}; };
class CaseStmt : public SwitchCase { class CaseStmt : public SwitchCase {
@ -439,6 +440,7 @@ class CaseStmt : public SwitchCase {
Stmt* SubExprs[END_EXPR]; // The expression for the RHS is Non-null for Stmt* SubExprs[END_EXPR]; // The expression for the RHS is Non-null for
// GNU "case 1 ... 4" extension // GNU "case 1 ... 4" extension
SourceLocation CaseLoc; SourceLocation CaseLoc;
virtual Stmt* v_getSubStmt() { return getSubStmt(); }
public: public:
CaseStmt(Expr *lhs, Expr *rhs, SourceLocation caseLoc) CaseStmt(Expr *lhs, Expr *rhs, SourceLocation caseLoc)
: SwitchCase(CaseStmtClass) { : SwitchCase(CaseStmtClass) {
@ -453,7 +455,6 @@ public:
Expr *getLHS() { return reinterpret_cast<Expr*>(SubExprs[LHS]); } Expr *getLHS() { return reinterpret_cast<Expr*>(SubExprs[LHS]); }
Expr *getRHS() { return reinterpret_cast<Expr*>(SubExprs[RHS]); } Expr *getRHS() { return reinterpret_cast<Expr*>(SubExprs[RHS]); }
Stmt *getSubStmt() { return SubExprs[SUBSTMT]; } Stmt *getSubStmt() { return SubExprs[SUBSTMT]; }
virtual Stmt* v_getSubStmt() { return getSubStmt(); }
const Expr *getLHS() const { const Expr *getLHS() const {
return reinterpret_cast<const Expr*>(SubExprs[LHS]); return reinterpret_cast<const Expr*>(SubExprs[LHS]);
} }
@ -467,8 +468,13 @@ public:
void setRHS(Expr *Val) { SubExprs[RHS] = reinterpret_cast<Stmt*>(Val); } void setRHS(Expr *Val) { SubExprs[RHS] = reinterpret_cast<Stmt*>(Val); }
virtual SourceRange getSourceRange() const { virtual SourceRange getSourceRange() const {
return SourceRange(CaseLoc, SubExprs[SUBSTMT]->getLocEnd()); // Handle deeply nested case statements with iteration instead of recursion.
const CaseStmt *CS = this;
while (const CaseStmt *CS2 = dyn_cast<CaseStmt>(getSubStmt()))
CS = CS2;
return SourceRange(CaseLoc, CS->getSubStmt()->getLocEnd());
} }
static bool classof(const Stmt *T) { static bool classof(const Stmt *T) {
return T->getStmtClass() == CaseStmtClass; return T->getStmtClass() == CaseStmtClass;
@ -486,12 +492,12 @@ public:
class DefaultStmt : public SwitchCase { class DefaultStmt : public SwitchCase {
Stmt* SubStmt; Stmt* SubStmt;
SourceLocation DefaultLoc; SourceLocation DefaultLoc;
virtual Stmt* v_getSubStmt() { return getSubStmt(); }
public: public:
DefaultStmt(SourceLocation DL, Stmt *substmt) : DefaultStmt(SourceLocation DL, Stmt *substmt) :
SwitchCase(DefaultStmtClass), SubStmt(substmt), DefaultLoc(DL) {} SwitchCase(DefaultStmtClass), SubStmt(substmt), DefaultLoc(DL) {}
Stmt *getSubStmt() { return SubStmt; } Stmt *getSubStmt() { return SubStmt; }
virtual Stmt* v_getSubStmt() { return getSubStmt(); }
const Stmt *getSubStmt() const { return SubStmt; } const Stmt *getSubStmt() const { return SubStmt; }
SourceLocation getDefaultLoc() const { return DefaultLoc; } SourceLocation getDefaultLoc() const { return DefaultLoc; }