[cfe-commits] [PATCH] [msvc] Honor %INCLUDE%. It should know essential search paths with vcvarsall.bat.

NAKAMURA Takumi geek4civic at gmail.com
Mon Mar 12 17:08:09 PDT 2012


Thanks, Michael. Committed in r152589.

2012/3/13 Michael Spencer <bigcheesegs at gmail.com>:
> The patch looks fine, the only thing I worry about is this code not
> being accessible from a MinGW built Clang. I'd rather we restructure
> the surrounding code to be enabled if we are both hosted on Windows,
> and targeting Win32.

I thought, last night, it would be useless that clang should behave
like cl.exe on mingw, even with "clang -target win32".

I suppose mingw users may specify individual include paths via command
line. The case "clang -target win32" can be considered as "cross
build".

But I rethink it might be useful as far as an user knows he is working
with ms headers.
For now, I leave _MSC_VER. In near future, I will reconfirm
"clang.exe" can behave similar among
{i686|x86_64}-{mingw32|mingw-w64|msvc}. Then I could prune _MSC_VER.

The conditioned block should be compiled also on mingw, that should make sense.

...Takumi



More information about the cfe-commits mailing list