[cfe-commits] r69511 - /cfe/trunk/lib/Sema/ParseAST.cpp

Sebastian Redl sebastian.redl at getdesigned.at
Sun Apr 19 11:06:17 PDT 2009


Chris Lattner wrote:
>
> On Apr 19, 2009, at 1:02 AM, Sebastian Redl wrote:
>
>>
>> On Sun, 19 Apr 2009 05:30:08 -0000, Chris Lattner <sabre at nondot.org>
>> wrote:
>>> Author: lattner
>>> Date: Sun Apr 19 00:30:08 2009
>>> New Revision: 69511
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=69511&view=rev
>>> Log:
>>> rearrange #include order.
>>>
>>
>> I think this deserves a comment to explain why we need Sema.h down
>> there.
>
> The problem is... I don't know!
I was thinking along the lines of, "Fucking stupid MSVC gets confused if
Sema.h is further up". Maybe without the profanity. :-)
Just something that makes people think twice before rearranging them
again, or at least remember this spot if MSVC again invents members.

Sebastian



More information about the cfe-commits mailing list