On Oct 30, 2007, at 1:28 PM, Chris Lattner wrote: >> >> + while (true) { >> + if (ParenExpr* P = dyn_cast<ParenExpr>(E)) >> + E = P->getSubExpr(); >> + else >> + break; >> + } > > How about: > > while (ParenExpr* P = dyn_cast<ParenExpr>(E)) > E = P->getSubExpr(); > > ? > Of course. I don't know what I was thinking.