[cfe-commits] r71589 - /cfe/trunk/lib/Sema/SemaDecl.cpp

[Chris Lattner]

On May 12, 2009, at 9:20 PM, Eli Friedman wrote:

> On Tue, May 12, 2009 at 1:55 PM, Chris Lattner <sabre at nondot.org>  
> wrote:
>> Author: lattner
>> Date: Tue May 12 15:55:07 2009
>> New Revision: 71589
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=71589&view=rev
>> Log:
>> Method who have definitions in structs/classes are semantically  
>> inline.
>> Per the FIXME, it might be interesting to track whether the inline  
>> keyword
>> was also used on the method, but for now we don't do this.   
>> Testcase pending.
>
> Did you actually test this?  As far as I can tell, it's impossible for
> IsFunctionDefinition to be true along this codepath.

Yep, you're right.  I tested it with other changes in my tree.  I've  
ripped it out for now, thanks Eli.

-Chris