[Mesa-dev] [PATCH] spirv: Silence a couple of warnings
Ian Romanick
idr at freedesktop.org
Thu Dec 6 00:05:49 UTC 2018
These look good to me. I don't remember whether or not that kind of
initializer makes MSCV happy, but meh.
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
On 12/05/2018 01:13 PM, Jason Ekstrand wrote:
> They occur when building with clang:
>
> [1/40] Compiling C object 'src/compiler/nir/s...piler at nir@@nir at sta/.._spirv_vtn_glsl450.c.o'.
> ../src/compiler/spirv/vtn_glsl450.c:845:39: warning: implicit conversion from enumeration type 'SpvOp' (aka 'enum SpvOp_') to different enumeration type 'enum GLSLstd450' [-Wenum-conversion]
> handle_glsl450_interpolation(b, ext_opcode, w, count);
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^~~~~~~~~~
> 1 warning generated.
> [2/40] Compiling C object 'src/compiler/nir/s...iler at nir@@nir at sta/.._spirv_spirv_to_nir.c.o'.
> ../src/compiler/spirv/spirv_to_nir.c:2167:30: warning: suggest braces around initialization of subobject [-Wmissing-braces]
> nir_tex_src none = {0};
> ^
> {}
> ../src/compiler/spirv/spirv_to_nir.c:2167:30: warning: suggest braces around initialization of subobject [-Wmissing-braces]
> nir_tex_src none = {0};
> ^
> {}
> 2 warnings generated.
> ---
> src/compiler/spirv/spirv_to_nir.c | 2 +-
> src/compiler/spirv/vtn_glsl450.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c
> index 22efaa276d9..2ce2cbd1c4d 100644
> --- a/src/compiler/spirv/spirv_to_nir.c
> +++ b/src/compiler/spirv/spirv_to_nir.c
> @@ -2164,7 +2164,7 @@ vtn_handle_texture(struct vtn_builder *b, SpvOp opcode,
> (*p++) = vtn_tex_src(b, w[idx++], nir_tex_src_offset);
>
> if (operands & SpvImageOperandsConstOffsetsMask) {
> - nir_tex_src none = {0};
> + nir_tex_src none = { .src_type = 0 };
> gather_offsets = vtn_ssa_value(b, w[idx++]);
> (*p++) = none;
> }
> diff --git a/src/compiler/spirv/vtn_glsl450.c b/src/compiler/spirv/vtn_glsl450.c
> index b54aeb9b217..dcfc85a579c 100644
> --- a/src/compiler/spirv/vtn_glsl450.c
> +++ b/src/compiler/spirv/vtn_glsl450.c
> @@ -842,7 +842,7 @@ vtn_handle_glsl450_instruction(struct vtn_builder *b, SpvOp ext_opcode,
> case GLSLstd450InterpolateAtCentroid:
> case GLSLstd450InterpolateAtSample:
> case GLSLstd450InterpolateAtOffset:
> - handle_glsl450_interpolation(b, ext_opcode, w, count);
> + handle_glsl450_interpolation(b, (enum GLSLstd450)ext_opcode, w, count);
> break;
>
> default:
>
More information about the mesa-dev
mailing list