<div class="gmail_quote">On Fri, Sep 9, 2011 at 9:49 PM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Ah, nevermind.  I misread the patch.</blockquote></div><br><div>Whew! Ok. Yea, I looked at that, and didn't intend to change that. Just to make sure I *did* understand this correctly, my impression was that this is controlling how we build up the parent block, prior to any splitting done by the introduction of cases, defaults, or labels. We need to see all the destructors and other delayed-effect inducing constructs.</div>
<div><br></div><div>That said, maybe we could do this when we visit the various statements? Essentially try to do the DeclStmt processing at the start of each of those visit methods? It would duplicate the logic here, but would result in only walking the chains of these once. Thoughts?</div>