Re: [Mesa-dev] [PATCH] spirv: Fix loading an entire block at once.

2017-12-08 Thread Jason Ekstrand
On Fri, Dec 8, 2017 at 9:36 AM, Emil Velikov wrote: > On 3 December 2017 at 14:40, Bas Nieuwenhuizen > wrote: > > There is no chain, so checking the length ends with a SEGFAULT. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103579 > > Cc: > I'm not 100% sure if a missing chain

Re: [Mesa-dev] [PATCH] spirv: Fix loading an entire block at once.

2017-12-08 Thread Emil Velikov
On 3 December 2017 at 14:40, Bas Nieuwenhuizen wrote: > There is no chain, so checking the length ends with a SEGFAULT. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103579 > Cc: I'm not 100% sure if a missing chain does not hint to an issue elsewhere? Patch does what it says on th

[Mesa-dev] [PATCH] spirv: Fix loading an entire block at once.

2017-12-03 Thread Bas Nieuwenhuizen
There is no chain, so checking the length ends with a SEGFAULT. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103579 Cc: --- src/compiler/spirv/vtn_variables.c | 58 -- 1 file changed, 30 insertions(+), 28 deletions(-) diff --git a/src/compiler/spir