https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #15 from Grimdoll ---
Thank you all very much, it solved my problem, I will write down this command
for the future.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
__
https://bugs.freedesktop.org/show_bug.cgi?id=88967
Grimdoll changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Previously, we special cased FB writes and URB writes in the register
allocation code. What we really wanted was to handle any message with
EOT set.
This saves us from extending the list with new opcodes in the future.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs_reg_all
This helper function basically just checks inst->eot, but also asserts
that only opcodes we expect to terminate threads have EOT set. As far
as I'm aware, we've never had such a bug.
Removing it means that we don't have to extend the list for new opcodes.
Cherryview and Skylake introduce an optim
https://bugs.freedesktop.org/show_bug.cgi?id=86837
--- Comment #24 from Jeff ---
Will this be making it into a future Mesa release?
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.free
Hi,
On 02/05/2015 01:31 AM, Kenneth Graunke wrote:
On Wednesday, February 04, 2015 01:52:57 PM Carl Worth wrote:
From: Kristian Høgsberg
This code provides for an on-disk cache of objects. Objects are stored
and retrieved (in ~/.cache/mesa) via names that are arbitrary 20-byte
sequences, (int
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #16 from Grimdoll ---
Created attachment 113190
--> https://bugs.freedesktop.org/attachment.cgi?id=113190&action=edit
Rangers with xorg driver
Can someone tell what is wrong with it? With proprietary driver all works fine.
--
You
https://bugs.freedesktop.org/show_bug.cgi?id=88967
Grimdoll changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|NOTABUG
Francisco Jerez writes:
> This is the first part of a series meant to improve our usage of the L3 cache.
> Currently it's far from ideal since the following objects aren't taking any
> advantage of it:
> - Pull constants (i.e. UBOs and demoted uniforms)
> - Buffer textures
> - Shader scratch s
On Thursday, January 15, 2015 02:05:18 AM Kenneth Graunke wrote:
> We were incorrectly attributing VS time to FS8 on Gen8+, which now use
> fs_visitor for vertex shaders.
>
> We don't hit this for geometry shaders yet, but we may as well add
> support now - the fix is obvious, and we'll just forge
Kenneth Graunke writes:
> On Thursday, January 15, 2015 02:05:18 AM Kenneth Graunke wrote:
>> We were incorrectly attributing VS time to FS8 on Gen8+, which now use
>> fs_visitor for vertex shaders.
>>
>> We don't hit this for geometry shaders yet, but we may as well add
>> support now - the fix
This test breaks !OpenGL 1_1.infinite-spot-light on broadwell.
Probe color at (0,0)
Expected: 0.50 0.00 0.00 1.00
Observed: 0.00 0.00 0.00 1.00
-Mark
Matt Turner writes:
> Prevents piglit regressions from the next patch.
> ---
> src/mesa/drivers/dri/i965/br
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #17 from Pavel Ondračka ---
(In reply to Grimdoll from comment #16)
> Created attachment 113190 [details]
> Rangers with xorg driver
>
> Can someone tell what is wrong with it? With proprietary driver all works
> fine.
There are som
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #18 from Pavel Ondračka ---
Ah, sorry about the llvm confusion, I didn't realize its r600, so it doesn't
need llvm for graphics.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for th
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #19 from Grimdoll ---
1) It is true, but this thread have some info about my system, and peopole no
need to ask it twice. And I though that second bug had relation to first one.
2) All asked info about system is in this thread
3) Name
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/ir_print_visitor.cpp | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/glsl/ir_print_visitor.cpp b/src/glsl/ir_print_visitor.cpp
index bd39805..3600827 100644
--- a/src/glsl/ir_print_visitor.cpp
+++ b/src/glsl/ir_print
From: Dave Airlie
(was: add double support)
Signed-off-by: Dave Airlie
---
src/glsl/ir_builder.cpp | 23 +++
src/glsl/ir_builder.h | 5 +
2 files changed, 28 insertions(+)
diff --git a/src/glsl/ir_builder.cpp b/src/glsl/ir_builder.cpp
index a2f6f29..37bbffa 100644
-
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/ir.cpp | 104 ++--
src/glsl/ir.h | 21
src/glsl/ir_validate.cpp| 61 ---
src/mesa/program/ir_to_mesa.cpp | 10
4 files cha
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/link_uniforms.cpp | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index de2f6c9..0db70d5 100644
--- a/src/glsl/link_uniforms.cpp
+++ b/src/glsl/link_unifo
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/ir_clone.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glsl/ir_clone.cpp b/src/glsl/ir_clone.cpp
index dffa578..5c7279c 100644
--- a/src/glsl/ir_clone.cpp
+++ b/src/glsl/ir_clone.cpp
@@ -327,6 +327,7 @@ ir_constant::clone(v
From: Dave Airlie
(was: add double support)
Signed-off-by: Dave Airlie
---
src/glsl/ir_constant_expression.cpp | 234 +++-
1 file changed, 202 insertions(+), 32 deletions(-)
diff --git a/src/glsl/ir_constant_expression.cpp
b/src/glsl/ir_constant_expression.cpp
I wanted to try if this could be split into smaller chunks to aid
review. Only compile tested (each step compiles).
Dave Airlie (17):
glsl: Add double builtin type (was: add double support)
glsl: Add double builtin type generation (was: add double support)
glsl: Uniform linking support for d
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/opt_constant_propagation.cpp | 3 +++
src/glsl/opt_minmax.cpp | 13 +
2 files changed, 16 insertions(+)
diff --git a/src/glsl/opt_constant_propagation.cpp
b/src/glsl/opt_constant_propagation.cpp
index c334e12
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/builtin_type_macros.h | 16 ++
src/glsl/glsl_parser_extras.h | 5 ++
src/glsl/glsl_types.cpp| 109 +
src/glsl/glsl_types.h | 18 ++-
4 files changed, 125 insertions(+)
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/lower_ubo_reference.cpp | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/src/glsl/lower_ubo_reference.cpp b/src/glsl/lower_ubo_reference.cpp
index 43dd067..e539491 100644
--- a/src/glsl/lower_ubo_referen
From: Dave Airlie
Perhaps squash this with previous.
Signed-off-by: Dave Airlie
---
src/glsl/builtin_types.cpp | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/glsl/builtin_types.cpp b/src/glsl/builtin_types.cpp
index 10fac0f..fef86df 100644
--- a/src/gls
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/glsl_parser.yy | 33 +
1 file changed, 29 insertions(+), 4 deletions(-)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index 7fb8c38..596e432 100644
--- a/src/glsl/glsl_parser.yy
+++ b/src
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/link_uniform_initializers.cpp | 7 ++-
src/glsl/link_varyings.cpp | 3 ++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/glsl/link_uniform_initializers.cpp
b/src/glsl/link_uniform_initializers.cpp
in
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/ast.h | 2 ++
src/glsl/ast_function.cpp | 67 +
src/glsl/ast_to_hir.cpp | 38 +--
src/glsl/glsl_parser_extras.cpp | 4 +++
4 files changed, 96 i
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/glsl_lexer.ll | 42 ++
1 file changed, 38 insertions(+), 4 deletions(-)
diff --git a/src/glsl/glsl_lexer.ll b/src/glsl/glsl_lexer.ll
index 57c46be..de58e73 100644
--- a/src/glsl/glsl_lexer.ll
+++
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/loop_controls.cpp | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/glsl/loop_controls.cpp b/src/glsl/loop_controls.cpp
index 1c1d34f..9a99c21 100644
--- a/src/glsl/loop_controls.cpp
+++ b/src/g
From: Dave Airlie
This adds the guts of the fp64 implementation to the GLSL compiler.
- builtin double types
- double constant support
- lexer parsing for double types (lf, LF)
- enforcing flat on double fs inputs
- double operations (d2f,f2d, pack/unpack, frexp - in 2 parts)
- ir builder bits.
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/glsl/ir_set_program_inouts.cpp | 24 +---
1 file changed, 21 insertions(+), 3 deletions(-)
diff --git a/src/glsl/ir_set_program_inouts.cpp
b/src/glsl/ir_set_program_inouts.cpp
index 97ead75..4aa402e 100644
--- a/src/glsl
On 01/24/2015 05:46 AM, Vinson Lee wrote:
> Fix build error.
>
> CCLD libGL.la
> libglapi.a(glapi_libglapi_la-glapi_gentable.o): In function
> `__glapi_gentable_NoOp':
> glapi_gentable.c:76: undefined reference to `backtrace'
>
> Signed-off-by: Vinson Lee
> ---
> src/mapi/glapi/gen/gl_ge
On 01/20/2015 01:31 AM, Matt Turner wrote:
> @@ -73,10 +72,14 @@ opt_cmod_propagation_local(fs_visitor *v, bblock_t *block)
> scan_inst->dst.reg == inst->src[0].reg &&
> scan_inst->dst.reg_offset == inst->src[0].reg_offset &&
> !scan_inst->is_partial_write(
Nothing special needs to be done.
Even though llvmpipe copies constant (ie uniform) buffers internally, the
application is suppose to flush and sync, so all should work.
All bufferstorage piglit tests pass.
---
src/gallium/drivers/llvmpipe/lp_screen.c | 3 ++-
1 file changed, 2 insertions(+), 1
Am 05.02.2015 um 15:33 schrieb Jose Fonseca:
> Nothing special needs to be done.
>
> Even though llvmpipe copies constant (ie uniform) buffers internally, the
> application is suppose to flush and sync, so all should work.
>
> All bufferstorage piglit tests pass.
> ---
> src/gallium/drivers/llvm
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #21 from Grimdoll ---
Created attachment 113204
--> https://bugs.freedesktop.org/attachment.cgi?id=113204&action=edit
glxinfo
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
On 05/02/15 15:07, Roland Scheidegger wrote:
Am 05.02.2015 um 15:33 schrieb Jose Fonseca:
Nothing special needs to be done.
Even though llvmpipe copies constant (ie uniform) buffers internally, the
application is suppose to flush and sync, so all should work.
All bufferstorage piglit tests pas
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #20 from Grimdoll ---
Created attachment 113203
--> https://bugs.freedesktop.org/attachment.cgi?id=113203&action=edit
dmseg
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the b
Topi, this is awesome! I wanted to do something like that last night,
but tiredness and laziness got in the way. Can I find these in git
form somewhere so that I'll be able to integrate when doing a resend?
(Also, I think it's fine to drop the "(was other patch)" in there.)
On Thu, Feb 5, 2015 at
On 02/05/2015 12:27 AM, Ilia Mirkin wrote:
From: Dave Airlie
This adds support for the new uniform interfaces
from ARB_gpu_shader_fp64.
v2:
support ARB_separate_shader_objects ProgramUniform*d* (Ian)
don't allow boolean uniforms to be updated (issue 15) (Ian)
v3: fix size_mul
v4: Teach unifor
On Thursday 05 February 2015, Michel Dänzer wrote:
> On 05.02.2015 12:48, Ilia Mirkin wrote:
> > Is there a benchmark that demonstrates this? I'd like to test it out
> > with nouveau.
>
> Mesa demos src/tests/streaming_rect on Kaveri (radeonsi):
>
> Unpatched: 42 frames in 1.023 seconds = 41.05
https://bugs.freedesktop.org/show_bug.cgi?id=88967
--- Comment #22 from Grimdoll ---
So what component I need to check for Gallium r600?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
___
On Thu, Feb 05, 2015 at 12:32:42AM -0800, Kenneth Graunke wrote:
> Previously, we special cased FB writes and URB writes in the register
> allocation code. What we really wanted was to handle any message with
> EOT set.
>
> This saves us from extending the list with new opcodes in the future.
>
Series Reviewed-by: Jordan Justen
On 2015-02-05 00:32:42, Kenneth Graunke wrote:
> Previously, we special cased FB writes and URB writes in the register
> allocation code. What we really wanted was to handle any message with
> EOT set.
>
> This saves us from extending the list with new opcodes
From: Marek Olšák
---
src/gallium/state_trackers/dri/dri_context.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trackers/dri/dri_context.c
b/src/gallium/state_trackers/dri/dri_context.c
index 84b8807..8ac81b7 100644
--- a/src/gallium/state_tracker
From: Christian König
Signed-off-by: Christian König
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 102 ++
src/gallium/winsys/radeon/drm/radeon_winsys.h | 11 +++
2 files changed, 113 insertions(+)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
b/src
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
I suggested in reply to 04/13 to squash the addition of
GLSL_TYPE_DOUBLE into 07/13, so with 07/13 split I'd squash it into
this patch.
With that fixed,
Reviewed-by: Matt Turner
_
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Perhaps squash this with previous.
No strong opinion. Either way,
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedeskt
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/link_uniforms.cpp | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
> index de2f6c9..
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/ir.cpp | 104
> ++--
> src/glsl/ir.h | 21
> src/glsl/ir_validate.cpp| 61 +++
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Maybe squash this somewhere? I'm not sure.
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Wed, Feb 04 2015, Tapani Pälli wrote:
> What would you think about changing this to use some defined maximum
> size (in MB)? I think for the user size is what matters and it could be
> a configurable option, number of items seems a bit vague and hard to
> predict (?)
Yes, changing over to a
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> (was: add double support)
I was going to suggest removing this from the commit subjects, but
just so you don't miss this one, remove it too. :)
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/ir_constant_expressio
Reviewed-by: Matt Turner
... assuming of course that these are in fact the only one you need
for the later lowering passes.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/lower_ubo_reference.cpp | 13 ++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/src/glsl/lower_ubo_reference.cpp
> b/src/glsl/lower_ubo_re
On Thu, Feb 05, 2015 at 10:23:27AM -0500, Ilia Mirkin wrote:
> Topi, this is awesome! I wanted to do something like that last night,
> but tiredness and laziness got in the way. Can I find these in git
> form somewhere so that I'll be able to integrate when doing a resend?
> (Also, I think it's fin
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
Presumably there's some code missing for d2b/b2d.
> src/glsl/ast.h | 2 ++
> src/glsl/ast_function.cpp | 67
> +
>
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/glsl_lexer.ll | 42 ++
> 1 file changed, 38 insertions(+), 4 deletions(-)
>
> diff --git a/src/glsl/glsl_lexer.ll b/src/glsl/glsl_l
I said in reply to the next patch that some hunks should go here.
Looking at this patch again, I'm not sure that's really the case.
Have a
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/ma
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/ir_set_program_inouts.cpp | 24 +---
> 1 file changed, 21 insertions(+), 3 deletions(-)
>
> diff --git a/src/glsl/ir_set_program_inouts.cpp
> b/src/gl
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/loop_controls.cpp | 19 +++
> 1 file changed, 15 insertions(+), 4 deletions(-)
>
> diff --git a/src/glsl/loop_controls.cpp b/src/glsl/loop_controls.cpp
> i
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> From: Dave Airlie
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/link_uniform_initializers.cpp | 7 ++-
> src/glsl/link_varyings.cpp | 3 ++-
> 2 files changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/l
On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
wrote:
> I wanted to try if this could be split into smaller chunks to aid
> review. Only compile tested (each step compiles).
Thanks a bunch for splitting these. It indeed makes it a ton easier to
review. I glanced at 07/13 and my eyes glazed over
On Thu, Feb 05, 2015 at 10:39:31AM -0800, Matt Turner wrote:
> Maybe squash this somewhere? I'm not sure.
I felt silly leaving it alone but I didn't really have a good squash
candidate for it. Perhaps somebody comes up with an idea, or we can just
toss a coin :)
___
On Thu, Feb 5, 2015 at 2:26 PM, Matt Turner wrote:
> On Thu, Feb 5, 2015 at 3:05 AM, Topi Pohjolainen
> wrote:
>> I wanted to try if this could be split into smaller chunks to aid
>> review. Only compile tested (each step compiles).
>
> Thanks a bunch for splitting these. It indeed makes it a ton
https://bugs.freedesktop.org/show_bug.cgi?id=88275
--- Comment #29 from Eugene ---
I tried stable MESA 10.1.3 with kernels 3.13.0-45 and 3.18.5. It works without
any problems. But installing latest MESA git immediately causes OpenGL not to
start with following errors:
libGL error: failed to crea
And what about removing the "RadeonProgram" page [1] ?
>From my point of view, this page depicts a wrong picture of the Radeon Mesa
driver. While I'm pretty sure most of the current and past games runs fine
today with r300g/r600g/radeonsi, this is not what one could think at a
first look when read
On Thu, Feb 5, 2015 at 2:47 PM, Benjamin Bellec wrote:
> And what about removing the "RadeonProgram" page [1] ?
>
> From my point of view, this page depicts a wrong picture of the Radeon Mesa
> driver. While I'm pretty sure most of the current and past games runs fine
> today with r300g/r600g/rade
Requires Evergreen/Cayman and radeon kernel module
2.41.0 or newer.
Signed-off-by: Glenn Kennard
---
Changes since v2:
* Fix failing arb_draw_indirect-vertexid piglit test cases.
* Ensure start_instance, base_vertex, index_offset are reset when
switching back to direct draws.
* Juggled some hea
---
src/glsl/nir/nir_validate.c | 62 -
1 file changed, 44 insertions(+), 18 deletions(-)
diff --git a/src/glsl/nir/nir_validate.c b/src/glsl/nir/nir_validate.c
index a34a017..13010ae 100644
--- a/src/glsl/nir/nir_validate.c
+++ b/src/glsl/nir/nir_valid
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_opt_gcm.c | 499 +
3 files changed, 502 insertions(+)
create mode 100644 src/glsl/nir/nir_opt_gcm.c
diff --git a/src/glsl/Makefile.sources b/src/glsl/Makefile.
---
src/glsl/nir/nir.h | 28 ++--
1 file changed, 22 insertions(+), 6 deletions(-)
diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h
index 4cb2e92..32c991a 100644
--- a/src/glsl/nir/nir.h
+++ b/src/glsl/nir/nir.h
@@ -416,15 +416,23 @@ typedef struct {
} nir_instr;
s
This index, unlike the others, has to be manually updated by passes that
want to use it.
---
src/glsl/nir/nir.c | 20
src/glsl/nir/nir.h | 4
2 files changed, 24 insertions(+)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
index 8ea7bb5..6758a72 100644
--- a/src/g
---
src/glsl/nir/nir.h | 3 +++
src/glsl/nir/nir_dominance.c | 9 -
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h
index 32c991a..8e5e686 100644
--- a/src/glsl/nir/nir.h
+++ b/src/glsl/nir/nir.h
@@ -1136,6 +1136,9 @@ typedef
---
src/glsl/nir/nir_validate.c | 87 -
1 file changed, 79 insertions(+), 8 deletions(-)
diff --git a/src/glsl/nir/nir_validate.c b/src/glsl/nir/nir_validate.c
index 13010ae..172406e 100644
--- a/src/glsl/nir/nir_validate.c
+++ b/src/glsl/nir/nir_valida
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index c3c268c..ba03291 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++ b/src/mesa/drivers/dri/i965/brw_f
This patch series provides an implementation of the global code motion
algorithm. The first 6 patches are fixups for pre-existing bugs in NIR,
the next 2 are add features needed by GCM, and the last two add GCM and
turn it on in i965. Shader-db numbers are as follows:
total NIR instructions in s
---
src/glsl/nir/nir_from_ssa.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/nir/nir_from_ssa.c b/src/glsl/nir/nir_from_ssa.c
index 2e7add3..2ac114d 100644
--- a/src/glsl/nir/nir_from_ssa.c
+++ b/src/glsl/nir/nir_from_ssa.c
@@ -277,7 +277,7 @@ get_parallel_copy_at_e
This is both simpler and more correct. The old code didn't properly index
load_const instructions.
---
src/glsl/nir/nir.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
index 10e6ed3..9a88bd3 100644
--- a/src
Previously, if you remved a CF node that still had instructions in it, none
of the use/def information from those instructions would get cleaned up.
Also, we weren't removing if statements from the if_uses of the
corresponding register or SSA def. This commit fixes both of these
problems
---
src/
On Wed, Feb 04 2015, Carl Worth wrote:
> First, when mapping the index file:
>
>/* FIXME: We map this shared, which is a start, but we need to think about
> * how to make it multi-process safe. */
>cache->index = (unsigned char *)
> mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_SHA
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/r300_reg.h| 22 ++
src/gallium/drivers/r300/r300_screen.c | 4 +---
src/gallium/drivers/r300/r300_state.c | 1 -
3 files changed, 3 insertions(+), 24 deletions(-)
diff --git a/src/gallium/drivers/r300/r300_re
Rather not assert on known factors, just print warning and use ZERO.
Signed-off-by: David Heidelberg
---
src/gallium/drivers/r300/r300_state_inlines.h | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/r300/r300_state_inlines.h
b/src/gallium/driv
https://bugs.freedesktop.org/show_bug.cgi?id=65534
Jason Ekstrand changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #5 from Jason Ekst
https://bugs.freedesktop.org/show_bug.cgi?id=65534
Kaveh changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |huax...@intel.com
|org
While we're cleaning this up, we should go a little farther - we
should also see if anything is using the SSA result of any of these
instructions, and if so make it point to an undef_instr instead to
avoid dangling pointers. This doesn't come up yet but it will when we
start cleaning up constant co
Reviewed-by: Connor Abbott
On Thu, Feb 5, 2015 at 5:28 PM, Jason Ekstrand wrote:
> This is both simpler and more correct. The old code didn't properly index
> load_const instructions.
> ---
> src/glsl/nir/nir.c | 26 --
> 1 file changed, 4 insertions(+), 22 deletions(-)
Other than one small comment below, 3 and 4 are
Reviewed-by: Connor Abbott
On Thu, Feb 5, 2015 at 5:28 PM, Jason Ekstrand wrote:
> ---
> src/glsl/nir/nir_validate.c | 87
> -
> 1 file changed, 79 insertions(+), 8 deletions(-)
>
> diff --git a/src/gl
Reviewed-by: Marek Olšák
Marek
On Thu, Feb 5, 2015 at 4:47 AM, Michel Dänzer wrote:
> From: Michel Dänzer
>
> The latter currently implies CPU access, so we have to avoid getting
> uncacheable memory.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88658
> Cc: "10.3 10.4"
> Signed-
Could use a comment here explaining that nir_instr_prev() may return
NULL above if the only instruction in the block is a jump instruction.
On Thu, Feb 5, 2015 at 5:29 PM, Jason Ekstrand wrote:
> ---
> src/glsl/nir/nir_from_ssa.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff -
This should go before the previous patch, or else that patch won't
work as intended, and also it confused me a little by using a behavior
(the returning NULL) before introducing it.
On Thu, Feb 5, 2015 at 5:29 PM, Jason Ekstrand wrote:
> ---
> src/glsl/nir/nir.h | 28 ++--
Reviewed-by: Connor Abbott
On Thu, Feb 5, 2015 at 5:29 PM, Jason Ekstrand wrote:
> ---
> src/glsl/nir/nir.h | 3 +++
> src/glsl/nir/nir_dominance.c | 9 -
> 2 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h
> index 32c991
On Thu, Feb 5, 2015 at 2:56 PM, David Heidelberg wrote:
> Signed-off-by: David Heidelberg
> ---
This is not a small clean up. This is removing a bunch of potentially
informative comments.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://
On Thu, Feb 5, 2015 at 2:56 PM, David Heidelberg wrote:
> Rather not assert on known factors, just print warning and use ZERO.
... because this allows me to and I cannot fix the
actual bug because of ...?
___
mesa-dev mailing list
mesa-dev@lists.freed
these days all (r600+,nv50+) modern HW support
PIPE_CAP_MAX_DUAL_SOURCE_RENDER_TARGETS Konsole output which r300 don't.
Instead of adding another expensive checks to Nine, this is way to
simply avoid it and do not cause assertion at no cost.
David
On 02/06/2015 12:59 AM, Matt Turner wrote:
On 02/06/2015 12:57 AM, Matt Turner wrote:
On Thu, Feb 5, 2015 at 2:56 PM, David Heidelberg wrote:
Signed-off-by: David Heidelberg
---
This is not a small clean up. This is removing a bunch of potentially
informative comments.
I checked sources & docs:
* all unsure comments has been verifi
1 - 100 of 168 matches
Mail list logo
