[llvm-commits] [PATCH] Fix bug with _MSC_VER define

John Garvin jgarvin at apple.com
Wed Aug 15 11:13:35 PDT 2012


It's not strictly necessary, but it would allow compiling with -Werror -Wundef.

John

On Aug 14, 2012, at 10:17 PM, Chris Lattner wrote:

> 
> On Aug 14, 2012, at 4:19 PM, John Garvin <jgarvin at apple.com> wrote:
> 
>> Support/Compiler.h was asking about the value of _MSC_VER without making sure it was defined first. Please review.
> 
> The patch isn't necessarily wrong, but why is it needed?  Use of undefined macros is specified to yield 0 by the C preprocessor.  I don't understand what this patch changes or fixes.
> 
> -Chris




More information about the llvm-commits mailing list