[cfe-commits] r101953 - /cfe/trunk/lib/AST/Decl.cpp

[Fariborz Jahanian]

On Apr 20, 2010, at 4:59 PM, Douglas Gregor wrote:

>
> On Apr 20, 2010, at 4:24 PM, Fariborz Jahanian wrote:
>
>>
>> On Apr 20, 2010, at 4:15 PM, Ted Kremenek wrote:
>>
>>> Author: kremenek
>>> Date: Tue Apr 20 18:15:35 2010
>>> New Revision: 101953
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=101953&view=rev
>>> Log:
>>> Specify linkage for Objective-C declarations.
>>
>> This makes me uncomfortable. This looks like something you need to
>> move the indexer forward (your next patch). Maybe you need to add
>> a new psuedo_linkage for your purpose.
>
> Can you explain a bit about why this makes you uncomfortable? From  
> my perspective, all of the Objective-C declarations have external  
> linkage since they are visible from multiple translation units.

Except for meta-data for the class (and its metaclass), othere meta- 
data have non-external linkage.
Content of all non-external metadata, however,  can be inspected with  
various APIs. In that respect they are 'external'.
But not wrt linkage.

- fariborz


>
>
> 	- Doug