Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-03 Thread Connor Abbott
On Fri, May 3, 2019 at 10:39 PM Jason Ekstrand wrote: > On Fri, May 3, 2019 at 3:29 PM Connor Abbott wrote: > >> FWIW, the reason I changed it away was to keep it consistent with the >> line directly above that uses the length (since the C array won't exist if >> it's length 0). Does that

Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-03 Thread Jason Ekstrand
On Fri, May 3, 2019 at 3:29 PM Connor Abbott wrote: > FWIW, the reason I changed it away was to keep it consistent with the line > directly above that uses the length (since the C array won't exist if it's > length 0). Does that convince you? > Nope. First off, if you take Dylan's suggestions

Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-03 Thread Connor Abbott
FWIW, the reason I changed it away was to keep it consistent with the line directly above that uses the length (since the C array won't exist if it's length 0). Does that convince you? On Fri, May 3, 2019 at 10:26 PM Jason Ekstrand wrote: > On Thu, May 2, 2019 at 3:51 PM Dylan Baker wrote: >

Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-03 Thread Jason Ekstrand
On Thu, May 2, 2019 at 3:51 PM Dylan Baker wrote: > Quoting Connor Abbott (2019-05-02 13:34:07) > > Just don't emit the transform array at all if there are no transforms > > for a state, and avoid trying to walk over it. > > --- > > Brian, does this build on Windows? I tested it on my shader-db

Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-02 Thread Brian Paul
On 05/02/2019 02:34 PM, Connor Abbott wrote: Just don't emit the transform array at all if there are no transforms for a state, and avoid trying to walk over it. --- Brian, does this build on Windows? I tested it on my shader-db on radeonsi. Yes, it compiles. Thanks! Tested-by: Brian Paul

Re: [Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-02 Thread Dylan Baker
Quoting Connor Abbott (2019-05-02 13:34:07) > Just don't emit the transform array at all if there are no transforms > for a state, and avoid trying to walk over it. > --- > Brian, does this build on Windows? I tested it on my shader-db > on radeonsi. > > --- > src/compiler/nir/nir_algebraic.py |

[Mesa-dev] [PATCH] nir/algebraic: Don't emit empty initializers for MSVC

2019-05-02 Thread Connor Abbott
Just don't emit the transform array at all if there are no transforms for a state, and avoid trying to walk over it. --- Brian, does this build on Windows? I tested it on my shader-db on radeonsi. --- src/compiler/nir/nir_algebraic.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)