r180741 - [analyzer] Change PathPieces to be a wrapper around an ilist of (through indirection) PathDiagnosticPieces.

[Jordan Rose]

On Apr 29, 2013, at 16:08 , Ted Kremenek <kremenek at apple.com> wrote:

> On Apr 29, 2013, at 4:02 PM, Jordan Rose <jordan_rose at apple.com> wrote:
>> I'm fairly sure flattening can still cause problems for when diagnostics are output twice, because PathDiagnosticPiece objects that contain path segments themselves really do need to be copied to make them independent. I don't think we really have a way to "share" the path pieces unless we have some kind of deepCloneNoLeaves() method.
> 
> I took a look at the flattening.  It really looks like we are either doing straight sharing or we conjure up new events.  The flattening essentially does a deep clone.

Ah, yes, I see. Other than that FIXME for macros, which should not be made worse by this.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130429/07f857cc/attachment.html>