[cfe-dev] Bug in LocationContextManager

[Zhongxing Xu]

Fortunately it was not causing bug because only AddPointer() and
AddInteger() are used in the profiling. But anyway they should be
virtual.

2009/11/3 Olaf Krzikalla <Olaf.Krzikalla at tu-dresden.de>:
> Hi,
>
> there seems to be a bug in LocationContextManager preventing it from doing
> what it is IMHO supposed to do: the member variable Contexts holds
> LocationContext (and profile them accordingly),  but the two insert
> functions profile the actual types. Thus if getStackFrame is called, always
> a new LocationContext is created. Making the Profile method in
> LocationContext virtual would solve the issue. Patch attached.
>
> Best
> Olaf Krzikalla
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>
>