https://bugs.freedesktop.org/show_bug.cgi?id=97285
Matthew Dawson changed:
What|Removed |Added
CC||matt...@mjdsystems.ca
--- Comment #5 fr
On Wednesday, August 10, 2016 2:22:29 PM PDT Jason Ekstrand wrote:
> On Tue, Aug 9, 2016 at 4:52 PM, Sirisha Gandikota <
> sirisha.gandik...@intel.com> wrote:
>
> > From: Sirisha Gandikota
> >
> > Several fixes have been added as part of this as listed below:
> >
> > 1) Fix the mask and add disas
https://bugs.freedesktop.org/show_bug.cgi?id=97285
Matthew Dawson changed:
What|Removed |Added
Status|NEEDINFO|REOPENED
--- Comment #4 from Matthew Da
Hi,
On Wednesday, 10 August 2016 15:19:06 CEST Nicolai Hähnle wrote:
> On 09.08.2016 22:12, Dave Airlie wrote:
> >>
> >> tbh, git submodules are more annoying than they need to be, and I'm
> >> not really terribly excited to use that for something that is a build
> >> dependency. Maybe just move
https://bugs.freedesktop.org/show_bug.cgi?id=97231
--- Comment #16 from Roland Scheidegger ---
(In reply to Jules Blok from comment #15)
> Created attachment 125671 [details]
> apitrace file llvmpipe
>
> Finally got an apitrace running on llvmpipe by using a VM. Sorry for the
> delay.
Thanks, w
https://bugs.freedesktop.org/show_bug.cgi?id=97285
--- Comment #3 from Kenneth Graunke ---
It might also be worth testing with and without multisampling...
--
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=97285
--- Comment #2 from Kenneth Graunke ---
On i965, both Dota 2 (Source 2) and Left 4 Dead 2 (Source 1) appear to be
working. I took screenshots from the same point in a Dota 2 tournament replay
both with and without the sRGB patches, and they look
On Wed, Aug 10, 2016 at 7:34 PM, Timothy Arceri <
timothy.arc...@collabora.com> wrote:
> Cause by 549222f5
>
s/Cause/Caused/ ?
Fixes the make check failure for me (haven't done a full piglit run, just
build/check/install test). Thanks for the quick turnaround.
--Aaron
>
> Cc: Aaron Watry
> -
Cause by 549222f5
Cc: Aaron Watry
---
src/compiler/glsl/tests/set_uniform_initializer_tests.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/compiler/glsl/tests/set_uniform_initializer_tests.cpp
b/src/compiler/glsl/tests/set_uniform_initializer_tests.cpp
index a36ffdc..9d41017 1
Hello,
I just got home and did a pull/build, and make check is failing as of a
very recent commit today...
${MESA_ROOT}/bin/test-driver: line 107: 5863 Segmentation fault (core
dumped) "$@" > $log_file 2>&1
FAIL: glsl/tests/uniform-initializer-test
PASS: glsl/glcpp/tests/glcpp-test
^Cmake[6
https://bugs.freedesktop.org/show_bug.cgi?id=97285
Ian Romanick changed:
What|Removed |Added
Status|NEW |NEEDINFO
CC|
On 08/10/2016 12:50 PM, Eric Anholt wrote:
> Connor Abbott writes:
>
>> On Wed, Aug 10, 2016 at 1:53 PM, Eric Anholt wrote:
>>> Kenneth Graunke writes:
>>>
On Haswell (GL 3.3):
total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
instructions in affected progra
On Wed, 2016-08-10 at 00:01 -0700, Eric Anholt wrote:
> Timothy Arceri writes:
>
> >
> > Builtins already have locations assigned so this shouldn't
> > changing anything. We want to call it earlier so we can tranform
>
> "change"
>
> Other than that, 1-4 are:
>
> Reviewed-by: Eric Anholt
>
https://bugs.freedesktop.org/show_bug.cgi?id=97285
Bug ID: 97285
Summary: Darkness in Dota 2 after Patch "Make Gallium's
BlitFramebuffer follow the GL 4.4 sRGB rules"
Product: Mesa
Version: unspecified
Hardware: Other
On Wed, Aug 10, 2016 at 9:44 AM, Michel Dänzer wrote:
> On 10/08/16 03:00 PM, Nicolas Boichat wrote:
>> eglMakeCurrent can also be used to change the active display. In that
>> case, we need to decrement ref_count of the previous display (possibly
>> destroying it), and increment it on the next di
On Wed, Aug 10, 2016 at 2:24 PM, Kenneth Graunke wrote:
> On Wednesday, August 10, 2016 10:02:12 AM PDT Erik Faye-Lund wrote:
>> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
>> wrote:
>> > On Haswell (GL 3.3):
>> >
>> > total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
>> >
On Wed, Aug 10, 2016 at 9:23 PM, Nicolai Hähnle wrote:
> Hi,
>
> this is a respin of the series which scans the shader's TGSI to determine
> which channels of an array are actually written to. Most of the st/mesa
> changes have become unnecessary. Most of the radeon-specific part stays
> the same.
On Tue, Aug 9, 2016 at 4:52 PM, Sirisha Gandikota
wrote:
> From: Sirisha Gandikota
>
> Several fixes have been added as part of this as listed below:
>
> 1) Fix the mask and add disassembler handling for STATE_DS, STATE_HS
> as the mask returned wrong values of the fields.
>
> 2) Fix the GEN_TYPE
On Tue, Aug 9, 2016 at 4:52 PM, Sirisha Gandikota
wrote:
> From: Kristian Høgsberg Kristensen
>
> The Aubinator tool is designed to help the driver developers in debugging
> the driver functionality by decoding the data in the .aub files.
> Primary Authors of this tool are Damien Lespiau
> and K
On Tue, Aug 9, 2016 at 4:52 PM, Sirisha Gandikota <
sirisha.gandik...@intel.com> wrote:
> From: Sirisha Gandikota
>
> Several fixes have been added as part of this as listed below:
>
> 1) Fix the mask and add disassembler handling for STATE_DS, STATE_HS
> as the mask returned wrong values of the
On Wed, Aug 10, 2016 at 11:14 AM, Ian Romanick wrote:
> On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> > This tries to move comparisons (a common source of boolean values)
> > closer to their first use. For GPUs which use condition codes,
> > this can eliminate a lot of temporary booleans and
Connor Abbott writes:
> On Wed, Aug 10, 2016 at 1:53 PM, Eric Anholt wrote:
>> Kenneth Graunke writes:
>>
>>> On Haswell (GL 3.3):
>>>
>>> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
>>> instructions in affected programs: 856541 -> 851642 (-0.57%)
>>> helped: 3157
>>> HUR
On 08/04/2016 06:51 PM, Brian Paul wrote:
On 08/04/2016 10:40 AM, Miklós Máté wrote:
On 07/14/2016 10:43 PM, Miklós Máté wrote:
On 07/07/2016 07:11 PM, Miklós Máté wrote:
On 06/26/2016 09:48 PM, Miklós Máté wrote:
Only include the ones that can be used by the shader.
This fixes texture coord
On Thu, Jul 28, 2016 at 9:55 PM, Connor Abbott wrote:
> On Thu, Jul 28, 2016 at 7:58 PM, Roland Scheidegger
> wrote:
>> Am 29.07.2016 um 00:35 schrieb Matt Turner:
>>> ---
>>> src/gallium/auxiliary/Makefile.sources | 1 +
>>> src/gallium/auxiliary/util/u_bitcast.h | 57
>>> +++
From: Nicolai Hähnle
Reviewed-by: Tom Stellard
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
i
From: Nicolai Hähnle
Also, prepare for using tgsi_array_info.
This also opens the door for properly handling allocation failures, but I'm
leaving that for a separate change.
---
src/gallium/drivers/radeon/radeon_llvm.h | 11 ++--
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 66
From: Nicolai Hähnle
They can lead to VM faults and worse, which goes against the GL robustness
promises.
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/galliu
From: Nicolai Hähnle
To have the same signature as get_array_range.
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_set
From: Nicolai Hähnle
---
src/gallium/drivers/radeon/radeon_llvm.h| 3 ++-
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 9 ++---
src/gallium/drivers/radeonsi/si_shader.c| 3 ++-
3 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/src/gallium/driver
From: Nicolai Hähnle
v2: take actual writemasks into account
---
src/gallium/drivers/radeon/radeon_llvm.h | 2 +
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 62 ++
2 files changed, 54 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/radeon/
From: Nicolai Hähnle
We can use the pointer stored in the temps array directly.
Reviewed-by: Tom Stellard
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers
From: Nicolai Hähnle
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 105 +
1 file changed, 66 insertions(+), 39 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
index 531a8fe..87fc0
From: Nicolai Hähnle
---
src/gallium/drivers/radeon/radeon_llvm.h | 4 +++
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 29 ++
src/gallium/drivers/radeonsi/si_shader.c | 19 +-
3 files changed, 34 insertions(+), 18 deletions(-)
diff -
From: Nicolai Hähnle
Doing the write-back of the temporary vector in radeon_llvm_emit_store makes
no sense.
This also allows us to get rid of get_alloca_for_array.
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 77 --
1 file changed, 28 insertions(+), 49 deletions(
From: Nicolai Hähnle
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 29 ++
1 file changed, 19 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
index 22ff18e..e4bfa
From: Nicolai Hähnle
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 73 --
1 file changed, 40 insertions(+), 33 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
index e4bfa74..994c7
From: Nicolai Hähnle
In the alloca'd array case, no longer create redundant and unused allocas
for the individual elements; create getelementptrs instead.
Reviewed-by: Tom Stellard
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 27 ++
1 file changed, 18 insertions
Hi,
this is a respin of the series which scans the shader's TGSI to determine
which channels of an array are actually written to. Most of the st/mesa
changes have become unnecessary. Most of the radeon-specific part stays
the same.
For one F1 2015 shader, it reduces the scratch size from 132096 t
From: Nicolai Hähnle
We can use the pointer stored in the temps array directly.
Reviewed-by: Tom Stellard
---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
b/src/gallium/drivers/r
From: Nicolai Hähnle
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 362559f..89e5c4d 100644
--- a/src/mesa/state_tracker/st_glsl_to_t
From: Nicolai Hähnle
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 76 ++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 17
2 files changed, 93 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index 98
On Wed, Aug 10, 2016 at 8:40 PM, Miklós Máté wrote:
> Thanks. I forgot to add that this should also be applied to the 12.0 stable
> branch.
OK. I've sent it to mesa-stable for inclusion into 12.0.
Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesk
From: Miklós Máté
The temporaries are allocated dynamically.
Signed-off-by: Miklós Máté
Signed-off-by: Marek Olšák
Cc: 12.0
---
src/mesa/state_tracker/st_atifs_to_tgsi.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_atifs_to_tgsi.c
b/src
Thanks. I forgot to add that this should also be applied to the 12.0
stable branch.
MM
On 08/10/2016 03:07 PM, Marek Olšák wrote:
Pushed, thanks.
Marek
On Mon, Aug 8, 2016 at 12:48 AM, Miklós Máté wrote:
The temporaries are allocated dynamically.
Signed-off-by: Miklós Máté
---
src/mesa
On 08/10/2016 11:24 AM, Kenneth Graunke wrote:
> On Wednesday, August 10, 2016 10:02:12 AM PDT Erik Faye-Lund wrote:
>> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
>> wrote:
>>> On Haswell (GL 3.3):
>>>
>>> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
>>> instructions i
On Wednesday, August 10, 2016 10:53:17 AM PDT Ian Romanick wrote:
> My most "recent" work in this area is in my appropriately named
> assume-CMS-in-precompile branch. I should just rename all my branches
> idr-got-distracted-#.
Thanks, I tried to look for your recent work on this, but in addition
On Wednesday, August 10, 2016 10:02:12 AM PDT Erik Faye-Lund wrote:
> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
> wrote:
> > On Haswell (GL 3.3):
> >
> > total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
> > instructions in affected programs: 856541 -> 851642 (-0.57%)
> >
> >
> > For now, this patch is
> >
> > Reviewed-by: Ian Romanick
>
I had a hard time parsing the title: "Turn -(b2f(a) + b2f(b) >= 0 into
!(a || b)" at first, until I saw the replacement instructions. You're
missing a ')' on the commit line. :)
___
mes
On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> This tries to move comparisons (a common source of boolean values)
> closer to their first use. For GPUs which use condition codes,
> this can eliminate a lot of temporary booleans and comparisons
> which reload the condition code register based on
On Wed, Aug 10, 2016 at 1:53 PM, Eric Anholt wrote:
> Kenneth Graunke writes:
>
>> On Haswell (GL 3.3):
>>
>> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
>> instructions in affected programs: 856541 -> 851642 (-0.57%)
>> helped: 3157
>> HURT: 113
>> LOST: 7
>> GAINED: 15
On Wednesday, August 10, 2016 10:58:49 AM PDT Ian Romanick wrote:
> On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> > On Haswell (GL 3.3):
> >
> > total instructions in shared programs: 6211485 -> 6211427 (-0.00%)
> > instructions in affected programs: 16260 -> 16202 (-0.36%)
> > helped: 25
> > H
On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6211427 -> 6210079 (-0.02%)
> instructions in affected programs: 219356 -> 218008 (-0.61%)
> helped: 567
> HURT: 132
>
> No spill/fill changes.
>
> LOST: 0
> GAINED: 4
>
> On Broad
Kenneth Graunke writes:
> Signed-off-by: Kenneth Graunke
> ---
> src/compiler/nir/nir_lower_io_to_scalar.c | 7 +--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/nir/nir_lower_io_to_scalar.c
> b/src/compiler/nir/nir_lower_io_to_scalar.c
> index f2345d5..8b
On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
> instructions in affected programs: 856541 -> 851642 (-0.57%)
> helped: 3157
> HURT: 113
> LOST: 7
> GAINED: 15
>
> On Broadwell (GL 4.4):
>
> total ins
On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6211485 -> 6211427 (-0.00%)
> instructions in affected programs: 16260 -> 16202 (-0.36%)
> helped: 25
> HURT: 37
>
> On Broadwell (GL 4.4):
>
> total instructions in shared programs:
My most "recent" work in this area is in my appropriately named
assume-CMS-in-precompile branch. I should just rename all my branches
idr-got-distracted-#.
On 08/09/2016 07:30 PM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6211678 -> 6211584 (-0.00%)
Kenneth Graunke writes:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
> instructions in affected programs: 856541 -> 851642 (-0.57%)
> helped: 3157
> HURT: 113
> LOST: 7
> GAINED: 15
>
> On Broadwell (GL 4.4):
>
> total instructions in shared prog
On Wed, Aug 10, 2016 at 09:43:24AM -0700, Nanley Chery wrote:
> On Tue, Aug 09, 2016 at 04:52:12PM -0700, Sirisha Gandikota wrote:
> > From: Sirisha Gandikota
> >
> > This is a patch series for adding the aubinator tool to the codebase.
> >
> > The aubinator tool is designed to help the driver d
On Wed, Aug 10, 2016 at 10:12:17AM -0700, Nanley Chery wrote:
> On Tue, Aug 02, 2016 at 10:00:29AM -0700, Jason Ekstrand wrote:
> > Signed-off-by: Jasosn Ekstrand
> > Cc: "12.0"
I just noticed a bug in this patch. With that fixed the Reviewed-by
will hold. Please see the note inline.
> > ---
>
On Tue, Aug 02, 2016 at 10:00:29AM -0700, Jason Ekstrand wrote:
> Signed-off-by: Jasosn Ekstrand
> Cc: "12.0"
> ---
> src/intel/vulkan/anv_meta_blit2d.c | 15 ++-
> 1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_meta_blit2d.c
> b/src/intel/vu
On Wed, Aug 10, 2016 at 4:34 PM, Nicolai Hähnle wrote:
> On 10.08.2016 13:05, Marek Olšák wrote:
>>
>> On Tue, Aug 9, 2016 at 12:56 PM, Nicolai Hähnle
>> wrote:
>>>
>>> On 07.08.2016 03:12, Marek Olšák wrote:
From: Marek Olšák
At compile time, each shader determines whic
On Tue, Aug 09, 2016 at 04:52:12PM -0700, Sirisha Gandikota wrote:
> From: Sirisha Gandikota
>
> This is a patch series for adding the aubinator tool to the codebase.
>
> The aubinator tool is designed to help the driver developers to debug
> the driver functionality by decoding the data in the
On Wed, Aug 10, 2016 at 5:48 PM, Jason Ekstrand wrote:
> On Aug 10, 2016 1:02 AM, "Erik Faye-Lund" wrote:
>>
>> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
>> wrote:
>> > On Haswell (GL 3.3):
>> >
>> > total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
>> > instructions in a
On Wed, Aug 10, 2016 at 5:40 PM, Jason Ekstrand wrote:
> On Aug 10, 2016 1:00 AM, "Erik Faye-Lund" wrote:
>>
>> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
>> wrote:
>> > On Haswell (GL 3.3):
>> >
>> > total instructions in shared programs: 6211678 -> 6211584 (-0.00%)
>> > instructions in a
On Tue, Aug 09, 2016 at 12:36:40PM +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
This is a great idea.
Reviewed-by: Tom Stellard
> ---
> src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 14 --
> 1 file changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/src/ga
On Tue, Aug 09, 2016 at 12:36:38PM +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> We can use the pointer stored in the temps array directly.
Reviewed-by: Tom Stellard
> ---
> src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 5 -
> 1 file changed, 5 deletions(-)
>
> diff --gi
On Tue, Aug 09, 2016 at 12:36:39PM +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> We can use the pointer stored in the temps array directly.
Reviewed-by: Tom Stellard
> ---
> src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 7 ---
> 1 file changed, 7 deletions(-)
>
> diff --
On Tue, Aug 09, 2016 at 12:36:37PM +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> In the alloca'd array case, no longer create redundant and unused allocas
> for the individual elements; create getelementptrs instead.
Reviewed-by: Tom Stellard
> ---
> .../drivers/radeon/radeon_setup_t
On Aug 10, 2016 1:02 AM, "Erik Faye-Lund" wrote:
>
> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
wrote:
> > On Haswell (GL 3.3):
> >
> > total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
> > instructions in affected programs: 856541 -> 851642 (-0.57%)
> > helped: 3157
> > HU
On Aug 10, 2016 1:00 AM, "Erik Faye-Lund" wrote:
>
> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke
wrote:
> > On Haswell (GL 3.3):
> >
> > total instructions in shared programs: 6211678 -> 6211584 (-0.00%)
> > instructions in affected programs: 18968 -> 18874 (-0.50%)
> > helped: 62
> > HURT:
FYI, I think I'm going to drop most of the gallium part of this series
and replace it by a pass over the resulting TGSI to look at which
components of the arrays are actually being used.
Nicolai
On 09.08.2016 12:36, Nicolai Hähnle wrote:
Hi,
this series was originally motivated by fixing a V
https://bugs.freedesktop.org/show_bug.cgi?id=96550
--- Comment #1 from war...@o2.pl ---
FYI:
ppl from Phoronix forums located the cause of the problem: gen_matypes.c is
passed to the compiler in order to generate assembly code in text form
(compiler switch: -S), but with -flto GCC doesn't output
https://bugs.freedesktop.org/show_bug.cgi?id=97231
Jules Blok changed:
What|Removed |Added
Attachment #125650|0 |1
is obsolete|
On 10.08.2016 01:34, Marek Olšák wrote:
Hi,
The first patch is pretty important and fixes the root cause of Overlord and
Witcher 2 hangs. The GLSL revert can be reverted after that.
I wonder what else this might fix. Anyway, nice catch! The series is
Reviewed-by: Nicolai Hähnle
Please re
On 10.08.2016 13:05, Marek Olšák wrote:
On Tue, Aug 9, 2016 at 12:56 PM, Nicolai Hähnle wrote:
On 07.08.2016 03:12, Marek Olšák wrote:
From: Marek Olšák
At compile time, each shader determines which ST_NEW flags should be set
at shader bind time.
This just sets the new field for all shader
Hi,
On Mon, 2016-05-23 at 14:18 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> A CTS test manages to generate this:
> GL45-CTS.shading_language_420pack.qualifier_order
>
> I cannot find definitive evidence in the spec that it isn't
> allowed. The specs mentions some things can't be used on
>
On 10.08.2016 06:22, Edward O'Callaghan wrote:
Well taking a step back here for a second. My initial thoughts are -
what, conceptually, is the primary user of addrlib from an extremely
high level view?
If the case is mesa3d then perhaps mesa should be treated as the
upstream from which other use
On 09.08.2016 22:12, Dave Airlie wrote:
tbh, git submodules are more annoying than they need to be, and I'm
not really terribly excited to use that for something that is a build
dependency. Maybe just move it into libdrm instead?
I've only had to use git submodules once with spice project, a
Pushed, thanks.
Marek
On Mon, Aug 8, 2016 at 12:48 AM, Miklós Máté wrote:
> The temporaries are allocated dynamically.
>
> Signed-off-by: Miklós Máté
> ---
> src/mesa/state_tracker/st_atifs_to_tgsi.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/state_t
For the series:
Reviewed-by: Marek Olšák
Marek
On Tue, Aug 9, 2016 at 12:38 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Avoid building all those store 0 / store undef instrucction pairs that
> end up getting removed anyway.
> ---
> src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
Pushed.
Marek
On Wed, Aug 10, 2016 at 1:16 PM, trevor.davenp...@gmail.com
wrote:
> I don't have commit access so someone else will need to push this out.
>
> Trevor
>
>
> On Aug 10, 2016 2:56 AM, "Marek Olšák" wrote:
>>
>> Reviewed-by: Marek Olšák
>>
>> Marek
>>
>> On Wed, Aug 10, 2016 at 5:28
This series is,
Reviewed-by: Edward O'Callaghan
On 08/10/2016 09:34 AM, Marek Olšák wrote:
> Hi,
>
> The first patch is pretty important and fixes the root cause of Overlord and
> Witcher 2 hangs. The GLSL revert can be reverted after that.
>
> Please review.
>
> Marek
>
> _
I don't have commit access so someone else will need to push this out.
Trevor
On Aug 10, 2016 2:56 AM, "Marek Olšák" wrote:
> Reviewed-by: Marek Olšák
>
> Marek
>
> On Wed, Aug 10, 2016 at 5:28 AM, Trevor Davenport
> wrote:
> > Since commit 6d7177f01b231e9fe79a558c28d2b562a218d7ea, radeonsi
>
On Tue, Aug 9, 2016 at 12:56 PM, Nicolai Hähnle wrote:
> On 07.08.2016 03:12, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> At compile time, each shader determines which ST_NEW flags should be set
>> at shader bind time.
>>
>> This just sets the new field for all shaders. The next commit will
On Tue, Aug 9, 2016 at 12:58 PM, Nicolai Hähnle wrote:
> I sent comments on patches 4 & 6. Apart from that, the series is
>
> Reviewed-by: Nicolai Hähnle
>
> On 07.08.2016 03:12, Marek Olšák wrote:
>>
>> PS: In order to make reviewing easier, all my patches have 10 lines
>> of contexts instead of
Reviewed-by: Marek Olšák
Marek
On Wed, Aug 10, 2016 at 5:28 AM, Trevor Davenport
wrote:
> Since commit 6d7177f01b231e9fe79a558c28d2b562a218d7ea, radeonsi
> would take a different path if info->indirect_params was not
> initialized properly. Nine was not initializating this field.
> ---
> src/
On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6208759 -> 6203860 (-0.08%)
> instructions in affected programs: 856541 -> 851642 (-0.57%)
> helped: 3157
> HURT: 113
> LOST: 7
> GAINED: 15
>
> On Broadwell (GL 4.4):
>
> to
On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke wrote:
> On Haswell (GL 3.3):
>
> total instructions in shared programs: 6211678 -> 6211584 (-0.00%)
> instructions in affected programs: 18968 -> 18874 (-0.50%)
> helped: 62
> HURT: 0
> LOST: 0
> GAINED: 4
>
> On Broadwell (GL 4.4):
>
> total ins
On 10/08/16 03:00 PM, Nicolas Boichat wrote:
> eglMakeCurrent can also be used to change the active display. In that
> case, we need to decrement ref_count of the previous display (possibly
> destroying it), and increment it on the next display.
>
> Also, old_dsurf/old_rsurf cannot be non-NULL if
Signed-off-by: Kenneth Graunke
---
src/compiler/nir/nir_lower_io_to_scalar.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/compiler/nir/nir_lower_io_to_scalar.c
b/src/compiler/nir/nir_lower_io_to_scalar.c
index f2345d5..8be75e4 100644
--- a/src/compiler/nir/nir_l
On Friday, August 5, 2016 4:27:03 PM PDT Eric Anholt wrote:
> vc4 wants to have per-scalar IO load/stores so that dead code elimination
> can happen on a more granular basis, which it has been doing in the
> backend using a multiplication by 4 of the intrinsic's driver_location.
> We can represent
Timothy Arceri writes:
> Builtins already have locations assigned so this shouldn't
> changing anything. We want to call it earlier so we can tranform
"change"
Other than that, 1-4 are:
Reviewed-by: Eric Anholt
and I'll see if I can get to some more tomorrow.
signature.asc
Description: P
92 matches
Mail list logo
