On Fri, Oct 3, 2014 at 10:03 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
Previously, we were allowing the register allocation code to do the
computation for us in ra_set_finalize. However, the runtime for this
computation is O(c^4 * g) where c is the number of classes and g is the
number
On Oct 3, 2014 11:06 PM, Connor Abbott cwabbo...@gmail.com wrote:
On Fri, Oct 3, 2014 at 10:03 PM, Jason Ekstrand ja...@jlekstrand.net
wrote:
Previously, we were allowing the register allocation code to do the
computation for us in ra_set_finalize. However, the runtime for this
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 85610fa..203e505 100644
---
From: Marek Olšák marek.ol...@amd.com
tgsi_shader_info contains everything we need.
---
src/gallium/drivers/radeonsi/si_shader.c | 22 +-
1 file changed, 5 insertions(+), 17 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_shader.c | 22 ++
src/gallium/drivers/radeonsi/si_state_draw.c | 4 ++--
2 files changed, 12 insertions(+), 14 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_state_draw.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 88fb2db..63edfde 100644
---
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_shader.c | 1 -
src/gallium/drivers/radeonsi/si_shader.h | 1 -
src/gallium/drivers/radeonsi/si_state_draw.c | 8 +---
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git
This is a cleanup series. It removes 147 lines of code. There are 2 goals:
- remove si_shader-inputs[] and outputs[] in favor of tgsi_shader_info
- don't use tgsi_parse_token, use tgsi_shader_info instead
On top of that, the shader key is shrunk a little in patches 2 and 3.
Please review.
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_shader.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 203e505..692d2a9 100644
---
From: Marek Olšák marek.ol...@amd.com
They were reinventing tgsi_shader_info. They are unused now.
radeon_llvm_context::load_input can be NULL if input fetching is implemented
in some other way.
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 3 +-
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_state_draw.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 63edfde..3fe8aab 100644
---
From: Marek Olšák marek.ol...@amd.com
No code in Mesa sets the usage mask to any other value.
The final mask is AND'ed with enable bits from the rasterizer state anyway.
If somebody implements setting usage masks in st/mesa, we can use
tgsi_shader_info to get it more easily.
This is a
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeonsi/si_state_draw.c | 22 +-
1 file changed, 13 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 3fe8aab..5f9e89f
From: Marek Olšák marek.ol...@amd.com
It's redundant now.
It led to a simplification in si_llvm_emit_streamout, because outidx == reg.
---
src/gallium/drivers/radeonsi/si_shader.c | 23 ++-
1 file changed, 6 insertions(+), 17 deletions(-)
diff --git
From: Marek Olšák marek.ol...@amd.com
Both cases are equivalent.
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.h | 2 +-
src/gallium/drivers/radeonsi/si_state.c | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git
From: Marek Olšák marek.ol...@amd.com
Those are going away.
---
src/gallium/drivers/radeonsi/si_shader.c | 10 +-
src/gallium/drivers/radeonsi/si_shader.h | 3 +++
src/gallium/drivers/radeonsi/si_state_draw.c | 4 ++--
3 files changed, 10 insertions(+), 7 deletions(-)
diff
From: Marek Olšák marek.ol...@amd.com
That code was really ugly.
---
src/gallium/drivers/radeonsi/si_shader.c | 39 +++-
1 file changed, 13 insertions(+), 26 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
From: Marek Olšák marek.ol...@amd.com
Written CLIPDIST outputs are simply disabled in PA_CL_VS_OUT_CNTL.
---
src/gallium/drivers/radeonsi/si_shader.c | 6 --
src/gallium/drivers/radeonsi/si_shader.h | 1 -
src/gallium/drivers/radeonsi/si_state.c | 8
3 files changed, 15
From: Marek Olšák marek.ol...@amd.com
This is the last use tgsi_parse_token in radeonsi.
It looks ugly because the code was re-indented, but there is really no change
in behavior.
---
src/gallium/drivers/radeonsi/si_shader.c | 132 ++-
1 file changed, 61
Hello list,
Here is the first 10.3 stable candidate. Currently we have
- 29 queued
- 13 nominated (outstanding)
- and 0 rejected patches
Take a look at section Mesa stable queue for more information.
Regressions
---
No regressions were spotted in swrast classic, swrast gallium and
On Fri, Oct 3, 2014 at 3:43 AM, Christian König deathsim...@vodafone.de wrote:
Am 03.10.2014 um 03:53 schrieb Ilia Mirkin:
On Thu, Oct 2, 2014 at 7:59 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 02/10/14 06:41, Ilia Mirkin wrote:
On Mon, Sep 29, 2014 at 8:33 PM, Emil Velikov
21 matches
Mail list logo