Looks good to me. Reviewed-by: Neha Bhende <bhen...@vmware.com> Regards, Neha
________________________________________ From: Brian Paul <bri...@vmware.com> Sent: Wednesday, September 11, 2019 11:23 AM To: mesa-dev@lists.freedesktop.org Cc: Neha Bhende; Charmaine Lee; aio.olive...@intel.com Subject: [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp IIRC, designated initializers are not legal C++. Fixes the MSVC build. Fixes: 83fd1e58 ("glsl/nir: Add and use a gl_nir_link() function") --- src/mesa/state_tracker/st_glsl_to_nir.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp b/src/mesa/state_tracker/st_glsl_to_nir.cpp index 280a778..d6a0264 100644 --- a/src/mesa/state_tracker/st_glsl_to_nir.cpp +++ b/src/mesa/state_tracker/st_glsl_to_nir.cpp @@ -688,7 +688,7 @@ st_link_nir(struct gl_context *ctx, */ if (shader_program->data->spirv) { static const gl_nir_linker_options opts = { - .fill_parameters = true, + true /*fill_parameters */ }; if (!gl_nir_link(ctx, shader_program, &opts)) return GL_FALSE; -- 1.8.5.6 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev