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

[Douglas Gregor]

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.

	- Doug