[cfe-commits] Adding interesting symbols during path diagnostic generation

Ted Kremenek kremenek at apple.com
Tue Mar 20 22:37:24 PDT 2012

On Mar 20, 2012, at 5:28 PM, Jordan Rose <jediknil at belkadan.com> wrote:

>>> Oh, it's not a visitor callback -- the interesting symbols and regions are stored in the BugReport. So it's just communication between the BugReport and the BugReporter.
>> Sorry, You are right.  I am a bit concerned with this method. It assumes that the symbols are never deleted from the set. Is this currently true?
> Yes, it is. If that changes, the implementation of this method will of course have to change. Is it worth proactively adding a change counter to BugReport?

I'm missing something.  Why do we need a change counter, or anything, in BugReport?  What we're suggesting should only impact some logic in BugReporter, and maybe something that allows us to roll back the BugReporterVisitor state, but both of these can be done without changing BugReport.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120320/732b5738/attachment.html>

More information about the cfe-commits mailing list