[cfe-commits] r60900 - in /cfe/trunk: include/clang/Parse/Ownership.h

[Cédric Venet]

steve naroff a écrit :
> On Dec 12, 2008, at 6:14 PM, Sebastian Redl wrote:
>
>   
>> Try /Za. However, if anything in LLVM includes <windows.h>, that won't
>> work, because in a case of grotesque idiocy by the various MS teams
>> involved in this, windows.h won't compile under /Za. MS never  
>> bothered to
>> implement #pragma system_header either.
>> Or, at least this was the case in VS.Net 2003. Maybe in 2005 it's
>> different.
>>
>>     
>
> Turning off the MS extensions with /Za works - "clangParse" now  
> builds:-)
>
> I imagine this will need to be added to the cmake config file for  
> clangParse...
>   

I commited a change to the cmake config fail to had /Za to all the clang 
libraries. I had to commit a small patch to llvm for fixing som 
dependent name lookup but it work for me. Perhaps it would be better to 
had /Za only to the libraries who need it? or to use the preprocessor 
solution, but I was under the impression that PP workaround where not 
liked in clang. Feel free to revert my changes.

> Thanks for your help on this!
>
> snaroff
>
>