[cfe-commits] [PATCH] Change cmp_XY x86/AVX builtin definition from static inline to macro in avxintrin.h

[Chris Lattner]

On Mar 28, 2011, at 1:15 AM, Syoyo Fujita wrote:

> Current implementation uses static inline definition for cmp_XY AVX
> instruction.
> With this clang cannot emit immediate value for 3rd argument where
> immediate value is requested.
> 
> This patch fixes the problem so that clang's codegen correctly emit
> immediate value for 3rd argument.

Please parenthesize all macro arguments (in particular use ", (c))" instead of ", c)".  Ok with that fix, thanks!

-Chris