[cfe-commits] r55211 - in /cfe/trunk/include/clang/Analysis/PathSensitive: Regions.h Store.h

[Chris Lattner]

On Aug 22, 2008, at 5:43 PM, Ted Kremenek wrote:

> Author: kremenek
> Date: Fri Aug 22 19:43:24 2008
> New Revision: 55211
>
> URL: http://llvm.org/viewvc/llvm-project?rev=55211&view=rev
> Log:
> Added Regions.h: Regions define abstract memory locations for the  
> static analyzer.

Hey Ted,

Can you please use a more specific name?  How about MemRegion or  
SAMemRegion?

Since this is just in the clang namespace, it is somewhat strange to  
give it a global name like this.  Alternatively, for things specific  
to the static analyzer, how about using a clang::SA namespace?

-Chris