[cfe-commits] r70293 - /cfe/trunk/lib/Analysis/CFRefCount.cpp

Ted Kremenek kremenek at apple.com
Tue Apr 28 19:21:06 PDT 2009


You are absolutely right.  I was mistaken.  Please re-apply.

On Apr 28, 2009, at 7:01 PM, Zhongxing Xu <xuzhongxing at gmail.com> wrote:

> 2009/4/29 Ted Kremenek <kremenek at apple.com>:
>> Hi Zhongxing,
>>
>> No we can't.  I'm still seeing symbolic regions wrapped in  
>> TypedViewRegions.
>
> Hi Ted,
>
> The removed code is only for non-symbolic regions wrapped in
> TypedViewRegions('void*',..). In that case, we need to remove the fake
> typed view. But now we already do not add 'void*' typed view to
> non-symbolic regions. For symbolic regions, we do need the
> TypedViewRegion that wraps them. So do we really need to revert this?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20090428/0811553d/attachment.html>


More information about the cfe-commits mailing list