[cfe-commits] r76515 - in /cfe/trunk: include/clang/Analysis/CallGraph.h include/clang/Index/Entity.h include/clang/Index/EntityHandler.h include/clang/Index/IndexProvider.h lib/Analysis/CallGraph.cpp lib/Index/Entity.cpp lib/Index/EntityImpl.h lib/Index/IndexProvider.cpp lib/Index/Program.cpp lib/Index/ProgramImpl.h test/Index/find-refs.c test/Index/foo.h test/Index/t1.c test/Index/t2.c tools/index-test/index-test.cpp

[Argyrios Kyrtzidis]

On Jul 21, 2009, at 9:16 AM, Douglas Gregor wrote:

>
> On Jul 20, 2009, at 5:07 PM, Argiris Kirtzidis wrote:
>
>> Author: akirtzidis
>> Date: Mon Jul 20 19:07:06 2009
>> New Revision: 76515
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=76515&view=rev
>> Log:
>> Change the semantics for Entity.
>>
>> Entity can now refer to declarations that are not visible outside
>> the translation unit.
>> It is a wrapper of a pointer union, it's either a Decl* for
>> declarations that don't
>> "cross" translation units, or an EntityImpl* which is associated
>> with the specific "visible" Decl.
>
> That seems reasonable. This will allow us to deal with static
> functions as well as fields, I assume.

The rational was for Entity to be a uniform way of referring to all  
declarations in the program,
the visible/internal split was unintuitive.

-Argiris