Re: [Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-30 Thread Jason Ekstrand
On Sat, Dec 30, 2017 at 3:57 PM, Grazvydas Ignotas wrote: > Hi, > > I don't know if it's the game's fault, but it appears this change broke > DOOM. > here is the offending spirv binary: > https://people.freedesktop.org/~notaz/doom_compute_spirv > Have you filed a bug? Please

Re: [Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-30 Thread Grazvydas Ignotas
Hi, I don't know if it's the game's fault, but it appears this change broke DOOM. here is the offending spirv binary: https://people.freedesktop.org/~notaz/doom_compute_spirv GraÅžvydas On Thu, Dec 7, 2017 at 6:12 PM, Jason Ekstrand wrote: > --- >

Re: [Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-11 Thread Ian Romanick
On 12/07/2017 02:26 PM, Jason Ekstrand wrote: > On Thu, Dec 7, 2017 at 11:54 AM, Michael Schellenberger Costa > > wrote: > > Hi Jason, > > > Am 07.12.2017 um 17:12 schrieb Jason Ekstrand: > > ---

Re: [Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-08 Thread Michael Schellenberger Costa
Hi Jason, Am 07.12.2017 um 17:12 schrieb Jason Ekstrand: --- src/compiler/spirv/vtn_variables.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index cf44ed3..8ce19ff 100644 ---

Re: [Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-07 Thread Jason Ekstrand
On Thu, Dec 7, 2017 at 11:54 AM, Michael Schellenberger Costa < mschellenbergerco...@googlemail.com> wrote: > Hi Jason, > > > Am 07.12.2017 um 17:12 schrieb Jason Ekstrand: > >> --- >> src/compiler/spirv/vtn_variables.c | 18 ++ >> 1 file changed, 14 insertions(+), 4

[Mesa-dev] [PATCH 3/8] spirv: Add basic type validation for OpLoad, OpStore, and OpCopyMemory

2017-12-07 Thread Jason Ekstrand
--- src/compiler/spirv/vtn_variables.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index cf44ed3..8ce19ff 100644 --- a/src/compiler/spirv/vtn_variables.c +++