From: Ian Romanick
textdata bss dec hex filename
7155954 256860 37332 7450146 71ae22 32-bit i965_dri.so before
7155858 256860 37332 7450050 71adc2 32-bit i965_dri.so after
6789395 328056 50704 7168155 6d609b 64-bit i965_dri.so before
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 34 +--
src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 10 +++-
src/mesa/drivers/dri/i915/intel_tex_layout.c | 3
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 3 ---
src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 8
2 files changed, 11 deletions(-)
diff --git
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 110 +-
1 file changed, 54 insertions(+), 56 deletions(-)
diff --git a/src/mesa/drivers/dri/i915/intel_mipmap_tree.c
From: Ian Romanick
Meta always sets the API to API_OPENGL_COMPAT, so the current API
setting is irrelevant.
textdata bss dec hex filename
7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so before
7154978 256860 37332 7449170 71aa52 32-bit
From: Ian Romanick
textdata bss dec hex filename
7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so before
7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so after
6788475 328056 50704 7167235 6d5d03 64-bit i965_dri.so before
From: Ian Romanick
Since i965 no longer uses this function for clearing color buffers,
there is no driver left that will ever support integer textures and use
_mesa_meta_glsl_Clear.
As a side note, the has_integer_textures check was rubbish anyway
because meta always
From: Ian Romanick
textdata bss dec hex filename
7155246 256860 37332 7449438 71ab5e 32-bit i965_dri.so before
7155058 256860 37332 7449250 71aaa2 32-bit i965_dri.so after
6788683 328056 50704 7167443 6d5dd3 64-bit i965_dri.so before
From: Ian Romanick
These are basically DSA versions of glLoadIdentity() and glLoadMatrix().
textdata bss dec hex filename
7155026 256860 37332 7449218 71aa82 32-bit i965_dri.so before
7155246 256860 37332 7449438 71ab5e 32-bit i965_dri.so
From: Ian Romanick
textdata bss dec hex filename
7155058 256860 37332 7449250 71aaa2 32-bit i965_dri.so before
7154994 256860 37332 7449186 71aa62 32-bit i965_dri.so after
6788611 328056 50704 7167371 6d5d8b 64-bit i965_dri.so before
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 5 ++---
src/mesa/drivers/dri/i915/intel_regions.c | 21 ++---
src/mesa/drivers/dri/i915/intel_regions.h | 5 ++---
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 5 -
src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 2 --
src/mesa/drivers/dri/i915/intel_tex.c | 4
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 9 -
1 file changed, 9 deletions(-)
diff --git a/src/mesa/drivers/dri/i915/intel_mipmap_tree.h
From: Ian Romanick
brw_blorp.c: In function ‘brw_blorp_clear_depth_stencil’:
brw_blorp.c:913:53: warning: unused parameter ‘partial_clear’
[-Wunused-parameter]
GLbitfield mask, bool partial_clear)
From: Ian Romanick
brw_blorp.c:1007:4: warning: ‘num_layers’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
blorp_clear_depth_stencil(, _surf, _surf,
^
From: Ian Romanick
In file included from brw_context.h:40:0,
from brw_blorp.c:32:
../../../../../src/intel/compiler/brw_compiler.h: In function
‘brw_stage_has_packed_dispatch’:
../../../../../src/intel/compiler/brw_compiler.h:1150:61: warning: unused
On 06/16/2017 06:25 AM, Rob Clark wrote:
> On Fri, Jun 16, 2017 at 9:18 AM, Emil Velikov
> wrote:
>> On 15 June 2017 at 21:47, Robert Foss wrote:
>>> From: Rob Herring
>>>
>>> This is required by freedreno at least for GLES3
Sent from my homecjf ghgrtyyytk th ttuithibygvgbuujjiiggti
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Same as the previous commit, but this one was split out because it's
a bit more complicated: this field is given as a pointer to a function,
so the function had to be changed as well, and the function was use in
a bunch of places, which needed updating as well.
Signed-off-by: Eric Engestrom
Signed-off-by: Eric Engestrom
---
src/egl/drivers/dri2/egl_dri2.c | 2 +-
src/egl/drivers/dri2/egl_dri2.h | 17 +
src/egl/drivers/dri2/platform_drm.c | 6 +++---
src/egl/drivers/dri2/platform_wayland.c | 20 ++--
On 06/15/2017 01:47 PM, Robert Foss wrote:
> From: Rob Herring
>
> This is required by freedreno at least for GLES3 support.
>
> See docs/patents.txt for information about turning this on for s/w
> renderers.
If I'm not mistaken (I'm not a lawyer, and I don't play one on TV),
It seems some of the tests I added require a a too long life-time of
the registers so that the actual aim of the patch is no longer
achieved.
Comments on the code and algorithm structure are welcome, but I have to
rethink the tests, because they seem to be too focused on corner-cases
that may
Hello Gert,
starting with short log from 'git am ...'
/opt/mesa> git am
~/Dokumente/Software/DRI-Mesa/Radeon/Mesa/mesa-st-glsl_to_tgsi-improved-temp-reg-lifetime-estimation.mbox
Wende an: mesa/st: glsl_to_tgsi move some helper classes to extra files
.git/rebase-apply/patch:561: new blank line
On 06/16/2017 01:37 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Since interface blocks are simply considered groupings of input variables,
> this is allowed.
>
> var->data.must_be_shader_input is now determined on-the-fly after lowering
> interface blocks,
This patch adds a class for tracking the life times of temporary registers
in the glsl to tgsi translation. The algorithm runs in three steps:
First, in order to minimize the number of needed memory allocations the
program is scanned to evaluate the number of scopes.
Then, the program is scanned
This patch ties in the new temporary register lifetime estiamtion and
rename mapping evaluation. In order to enable it, the evironment
variable MESA_GLSL_TO_TGSI_NEW_MERGE must be set.
Performance to compare between the current and the new implementation
were measured by running the shader-db in
For the new register renaming approach C++11 is wanted.
---
src/mesa/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/Makefile.am b/src/mesa/Makefile.am
index 53f311d2a9..3339926d93 100644
--- a/src/mesa/Makefile.am
+++ b/src/mesa/Makefile.am
@@ -101,7
The remapping evaluator first sorts the temporary registers ascending
based on their first life time instruction, and then uses a binary search
to find merge canidates.
For the initial sorting it uses std::sort because qsort is quite slow in
comparison. By removing the define USE_STL_SORT in
This patch adds a set of unit tests for the new lifetime tracker.
---
configure.ac | 1 +
src/mesa/Makefile.am | 2 +-
src/mesa/state_tracker/tests/Makefile.am | 41 +
.../tests/test_glsl_to_tgsi_lifetime.cpp
To prepare the implementation of a temp register lifetime tracker
some of the classes are moved into seperate header/implementation
files to make them accessible from other files.
Specifically these are:
class st_src_reg;
class st_dst_reg;
class glsl_to_tgsi_instruction;
struct
The patch adds tests for the register rename mapping evaluation.
---
.../tests/test_glsl_to_tgsi_lifetime.cpp | 71 --
1 file changed, 66 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/tests/test_glsl_to_tgsi_lifetime.cpp
Dear all,
following the comments of Emil and Nicolai I've updated the patch set.
Changes with respect to the old version are:
- split the changes into more patches
- correct formatting errors
- remove the use of the STL with one exception though:
since in st_glsl_to_tgsi.cpp std::sort is
Hello Dieter,
> W'll start with your attachment
> https://bugs.freedesktop.org/attachment.cgi?id=131683
This one is a dirty hack that works around the use of too many
registers mostly by allowing to allocate more than the allowed limit
and let the bytecode optimizer sort it out (there is also
Hello Gert, (hello Nicolai, ;-))
do you have some 'work in progress' ready.
Then I'll put my Turks XT back in and test LS2015 (Farming Simulator
2015) under Wine. It show missing details (the driver (the player) and
all the other people walking around) with current Mesa git code. I get
the
Hello Nicolai,
Am Sonntag, den 18.06.2017, 12:05 +0200 schrieb Nicolai Hähnle:
>
> > if HAVE_XLIB_GLX
> > SUBDIRS += drivers/x11
> > @@ -101,7 +101,7 @@ AM_CFLAGS = \
> > $(VISIBILITY_CFLAGS) \
> > $(MSVC2013_COMPAT_CFLAGS)
> > AM_CXXFLAGS = \
> > - $(LLVM_CFLAGS) \
> > +
On 06/15/2017 11:59 PM, Timothy Arceri wrote:
On 14/06/17 19:27, Samuel Pitoiset wrote:
It would be nice to have a no_error path for
_mesa_test_texobj_completeness() because this function doesn't
only test if the texture is complete.
Are you sure? I don't see any error checks in there.
On 16.06.2017 11:32, Gert Wollny wrote:
This patch adds new classes and tests to implement a tracker for the
life time of temporary registers for the register renaming stage of
glsl_to_tgsi. The tracker aims at estimating the shortest possible
life time for each register. The code base requires
37 matches
Mail list logo