[Mesa-dev] [Bug 65373] Rename the _mesa_lookup_enum_by_nr() function

2016-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65373 Timothy Arceri changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [Mesa-dev] [PATCH 1/2] i965: Use nir_lower_load_const_to_scalar().

2016-01-23 Thread Jason Ekstrand
On Jan 23, 2016 12:19 AM, "Kenneth Graunke" wrote: > > On Friday, January 22, 2016 11:54:22 PM PST Jason Ekstrand wrote: > > On Jan 21, 2016 4:37 PM, "Kenneth Graunke" wrote: > > > > > > I don't know why, but we never hooked up this pass Eric wrote. > > > Otherwise, you can end up with stupid sca

Re: [Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Marek Olšák
On Sat, Jan 23, 2016 at 8:02 PM, Kenneth Graunke wrote: > On Saturday, January 23, 2016 12:47:51 PM PST Marek Olšák wrote: >> From: Marek Olšák >> >> --- >> split-to-files.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/split-to-files.py b/split-to-files.py >> inde

Re: [Mesa-dev] [PATCH 0/6] Move nir/glsl to src/compiler, take 2

2016-01-23 Thread Jose Fonseca
On 22/01/16 16:43, Emil Velikov wrote: On 19 January 2016 at 22:36, Matt Turner wrote: On Mon, Jan 18, 2016 at 4:02 AM, Emil Velikov wrote: Hi all, Here is another attempt at splitting up nir, glsl and the rest. The gist is that we have three independent static libraries - compiler, nir and

Re: [Mesa-dev] The get/get_hash_params situation...

2016-01-23 Thread Ilia Mirkin
On Sat, Jan 23, 2016 at 1:56 PM, Kenneth Graunke wrote: > I'll be interested to see what you come up with. Maybe send a draft of > your idea so people can take a look before you bother to convert the > whole thing? My immediate thought is to create a hash table of enum -> { avail, v, pred, func

Re: [Mesa-dev] [PATCH v2] glsl: Enable debug prints for do_common_optimization

2016-01-23 Thread Kenneth Graunke
On Thursday, January 21, 2016 11:35:31 PM PST Jordan Justen wrote: > Signed-off-by: Jordan Justen > --- > > I tested a release build. I think v1 produced comparable code to > before the patch, but it was a bit larger. I found that this v2 code > would produce identical code to our current code

Re: [Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Kenneth Graunke
On Saturday, January 23, 2016 12:47:51 PM PST Marek Olšák wrote: > From: Marek Olšák > > --- > split-to-files.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/split-to-files.py b/split-to-files.py > index 151681e..14f19a4 100755 > --- a/split-to-files.py > +++ b/split

Re: [Mesa-dev] The get/get_hash_params situation...

2016-01-23 Thread Kenneth Graunke
On Saturday, January 23, 2016 9:49:24 AM PST Ilia Mirkin wrote: > ... sucks. Every change to that file is an exercise in pain. We want > more and more complex checks on all the various enums, as they tend to > appear in GL core with ext A or in GL ES with ext B but only if ES > > 3.1, and not if th

[Mesa-dev] [PATCH] mesa: fix MAX_FRAMEBUFFER_LAYERS query

2016-01-23 Thread Ilia Mirkin
ARB_framebuffer_no_attachments can be enabled in compat contexts, where there are no geometry shaders. But the spec still says that this enum should be exposed. This approach will also expose the query in plain GLES 3.1 contexts that don't have GL_OES_geometry_shader enabled, but the get system do

[Mesa-dev] [PATCH 3/3] si-report.py: process Max Waves

2016-01-23 Thread Marek Olšák
From: Marek Olšák --- si-report.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/si-report.py b/si-report.py index 12e16c3..c7fe1b5 100755 --- a/si-report.py +++ b/si-report.py @@ -61,6 +61,7 @@ class si_stats: ('code_size', 'Code Size', 'bytes'), ('lds

[Mesa-dev] [PATCH 2/3] si-report.py: fix parsing without asm

2016-01-23 Thread Marek Olšák
From: Marek Olšák --- si-report.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/si-report.py b/si-report.py index 3156639..12e16c3 100755 --- a/si-report.py +++ b/si-report.py @@ -176,6 +176,8 @@ class si_parser(object): match = si_parser.re_stats.match(msg)

[Mesa-dev] [PATCH 1/3] run: update the R600_DEBUG value

2016-01-23 Thread Marek Olšák
From: Marek Olšák --- run.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/run.c b/run.c index a91b18d..12f0ed6 100644 --- a/run.c +++ b/run.c @@ -361,7 +361,7 @@ main(int argc, char **argv) setenv("allow_glsl_extension_directive_midshader", "true", 1); setenv("sha

[Mesa-dev] [PATCH] radeonsi: fix shader precompilation for shader-db

2016-01-23 Thread Marek Olšák
From: Marek Olšák The addition of spi_shader_col_format killed all color outputs in precompiled shaders. --- src/gallium/drivers/radeonsi/si_state_shaders.c | 43 +++-- 1 file changed, 34 insertions(+), 9 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_shader

[Mesa-dev] The get/get_hash_params situation...

2016-01-23 Thread Ilia Mirkin
... sucks. Every change to that file is an exercise in pain. We want more and more complex checks on all the various enums, as they tend to appear in GL core with ext A or in GL ES with ext B but only if ES > 3.1, and not if the moon is waxing, etc. Any objections if I rewrite it to be not so anno

[Mesa-dev] [PATCH] mesa: invalidate framebuffer when changing parameters

2016-01-23 Thread Ilia Mirkin
This fixes dEQP-GLES31.functional.fbo.completeness.no_attachments When the width or height are 0, the framebuffer is incomplete. We may also not have been passing the new state down to the driver when the widths/heights changed. Signed-off-by: Ilia Mirkin --- Tested on nvc0, relying on some unr

Re: [Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Ilia Mirkin
On Sat, Jan 23, 2016 at 8:05 AM, Marek Olšák wrote: > On Sat, Jan 23, 2016 at 1:28 PM, Ilia Mirkin wrote: >> H... Because you can have >> >> // asdf >> #version 150? >> >> Unfortunately this would also catch >> >> // #version 450 >> #version 150 >> >> Would it work to put a ^ in the regex, i.

Re: [Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Marek Olšák
On Sat, Jan 23, 2016 at 1:28 PM, Ilia Mirkin wrote: > H... Because you can have > > // asdf > #version 150? > > Unfortunately this would also catch > > // #version 450 > #version 150 > > Would it work to put a ^ in the regex, i.e. r".*^#version", re.DOTALL? I don't know. I don't remember

Re: [Mesa-dev] [PATCH 2/2] split-to-files.py: fix parsing tessellation shaders

2016-01-23 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Sat, Jan 23, 2016 at 6:47 AM, Marek Olšák wrote: > From: Marek Olšák > > --- > split-to-files.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/split-to-files.py b/split-to-files.py > index 14f19a4..04dc867 100755 > --- a/split-to-files

Re: [Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Ilia Mirkin
H... Because you can have // asdf #version 150? Unfortunately this would also catch // #version 450 #version 150 Would it work to put a ^ in the regex, i.e. r".*^#version", re.DOTALL? -ilia On Sat, Jan 23, 2016 at 6:47 AM, Marek Olšák wrote: > From: Marek Olšák > > --- > split-t

[Mesa-dev] [PATCH 1/2] split-to-files.py: fix parsing when #version is not at the beginning

2016-01-23 Thread Marek Olšák
From: Marek Olšák --- split-to-files.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/split-to-files.py b/split-to-files.py index 151681e..14f19a4 100755 --- a/split-to-files.py +++ b/split-to-files.py @@ -71,7 +71,7 @@ def write_shader_test(filename, shaders): min_ver

[Mesa-dev] [PATCH 2/2] split-to-files.py: fix parsing tessellation shaders

2016-01-23 Thread Marek Olšák
From: Marek Olšák --- split-to-files.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/split-to-files.py b/split-to-files.py index 14f19a4..04dc867 100755 --- a/split-to-files.py +++ b/split-to-files.py @@ -88,9 +88,9 @@ def write_shader_test(filename, shaders):

Re: [Mesa-dev] [PATCH 7/7] radeonsi: emit rw_buffers for tes_shader only if tes_shader present

2016-01-23 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák On Sat, Jan 23, 2016 at 1:00 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > --- > src/gallium/drivers/radeonsi/si_descriptors.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/drivers/radeonsi/si_des

Re: [Mesa-dev] [PATCH 1/2] i965: Use nir_lower_load_const_to_scalar().

2016-01-23 Thread Kenneth Graunke
On Friday, January 22, 2016 11:54:22 PM PST Jason Ekstrand wrote: > On Jan 21, 2016 4:37 PM, "Kenneth Graunke" wrote: > > > > I don't know why, but we never hooked up this pass Eric wrote. > > Otherwise, you can end up with stupid scalarized code such as: > > > >vec4 ssa_7 = load_const (0.0, 0