[cfe-dev] Lazy bindings

Zhongxing Xu xuzhongxing at gmail.com
Wed Dec 17 21:49:39 PST 2008


On Thu, Dec 18, 2008 at 9:53 AM, Ted Kremenek <kremenek at apple.com> wrote:

> On Dec 17, 2008, at 4:22 AM, Zhongxing Xu wrote:
>
>  Hi Ted,
>>
>> This is the patch for lazy binding.
>>
>
> Hi Zhongxing,
>
> This is a huge patch, so I'm going to make a few passes over it.  Overall
> it looks very good.  I'm wondering if we can break out the changes to
> SymbolManager.[h,cpp] into a separate patch without compromising the current
> functionality of BasicStore.  It would also make your changes to RegionStore
> more isolated.
>


The SymbolManager part of the patch added const'ness to VarDecl* and
ParmVarDecl*. It will break the compilation if committed separately.
Essentially, it only adds one function:
  SymbolRef SymbolManager::getSymbol(const MemRegion* R);
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20081218/cc7c203d/attachment.html>


More information about the cfe-dev mailing list