Re: [Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-16 Thread Caio Marcelo de Oliveira Filho
> How about something simple like this instead: > > > diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp > b/src/mesa/state_tracker > index d6a0264..4f5acfd 100644 > --- a/src/mesa/state_tracker/st_glsl_to_nir.cpp > +++ b/src/mesa/state_tracker/st_glsl_to_nir.cpp > @@ -687,9 +687,14 @@

Re: [Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-12 Thread Brian Paul
On 09/11/2019 03:06 PM, Ian Romanick wrote: On 9/10/19 10:53 PM, Brian Paul wrote: 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,

Re: [Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-11 Thread Ian Romanick
On 9/10/19 10:53 PM, Brian Paul wrote: > 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(-) > >

Re: [Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-11 Thread Matt Turner
On Tue, Sep 10, 2019 at 10:54 PM Brian Paul wrote: > > 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

Re: [Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-11 Thread Neha Bhende
Looks good to me. Reviewed-by: Neha Bhende Regards, Neha From: Brian Paul 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

[Mesa-dev] [PATCH] st/nir: fix illegal designated initializer in st_glsl_to_nir.cpp

2019-09-10 Thread Brian Paul
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