[Mesa-dev] [PATCH] nir: no-op C99 _Pragma() with MSVC

Ian Romanick idr at freedesktop.org
Sat Nov 23 04:16:09 UTC 2019


On 11/22/19 6:49 PM, Brian Paul wrote:
> This fixes a build failure on MSVC.
> 
> BTW, it looks like clang supports _Pragma() but I don't know if it
> understands the "gcc unroll N" directive.

It probably doesn't, but that should be okay.  This just exists to speed
up the debug builds in the pre-merge CI.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> Signed-off-by: Brian Paul <brianp at vmware.com>
> ---
>  src/compiler/nir/nir_range_analysis.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/compiler/nir/nir_range_analysis.c b/src/compiler/nir/nir_range_analysis.c
> index df5d4da..d38bcc0 100644
> --- a/src/compiler/nir/nir_range_analysis.c
> +++ b/src/compiler/nir/nir_range_analysis.c
> @@ -218,6 +218,13 @@ analyze_constant(const struct nir_alu_instr *instr, unsigned src,
>   */
>  #define _______ unknown
>  
> +
> +/* MSVC doesn't have C99's _Pragma() */
> +#ifdef _MSC_VER
> +#define _Pragma(x)
> +#endif
> +
> +
>  #ifndef NDEBUG
>  #define ASSERT_TABLE_IS_COMMUTATIVE(t)                        \
>     do {                                                       \
> 



More information about the mesa-dev mailing list