Before this patch, the following code would not be optimized even though
the final two instructions were common to the then and else blocks:
(+f0) IF
MOV dst2 ...
MOV dst1 ...
MOV dst0 ...
ELSE
MOV dst3 ...
MOV dst1 ...
MOV dst0 ...
ENDIF
This commit extends the peephol
Removes if/endif and if/else/endif.
total instructions in shared programs: 1366420 -> 1356988 (-0.69%)
instructions in affected programs: 160818 -> 151386 (-5.87%)
---
Split mistakenly squashed patch.
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/brw_fs
On Tue, 2013-10-29 at 13:13 -0700, Eric Anholt wrote:
> Josh Klint writes:
>
> > Hi,
> >
> > As I understand it, Mesa is in charge of providing OpenGL drivers for
> > Intel graphics hardware running on Linux. I'm in the process of
> > porting our game engine over to Linux, and have been pleasant
On 29 October 2013 19:22, Anuj Phogat wrote:
>
>
>
> On Mon, Oct 28, 2013 at 5:10 PM, Paul Berry wrote:
>
>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>
>>> V2:
>>>- Update comments
>>>- Use fs_reg(0x) in AND instruction to get the 16 bit
>>> sample_mask.
>>>- Add a spec
On 29 October 2013 21:28, Paul Berry wrote:
> On 29 October 2013 17:16, Anuj Phogat wrote:
>
>>
>>
>>
>> On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry wrote:
>>
>>
>>> I think the right thing to do is to add:
>>>
>>> if (dispatch_width == 16)
>>> fail("...");
>>>
>>> to whatever parts of the v
On 29 October 2013 17:16, Anuj Phogat wrote:
>
>
>
> On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry wrote:
>
>
>> I think the right thing to do is to add:
>>
>> if (dispatch_width == 16)
>> fail("...");
>>
>> to whatever parts of the visitor you aren't confident will work properly
>> in SIMD16.
On Mon, Oct 28, 2013 at 5:10 PM, Paul Berry wrote:
> On 25 October 2013 16:45, Anuj Phogat wrote:
>
>> V2:
>>- Update comments
>>- Use fs_reg(0x) in AND instruction to get the 16 bit
>> sample_mask.
>>- Add a special backend instructions to compute sample_mask.
>>- Add a
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #8 from Bryce Harrington ---
Yep, that does the trick.
Tested-by: Bryce Harrington
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mes
From: Ian Romanick
This isn't going to be used in the actual implemenation of
glGetGraphicsResetStatus.
Signed-off-by: Ian Romanick
---
src/mesa/main/context.c | 1 -
src/mesa/main/getstring.c | 2 +-
src/mesa/main/mtypes.h| 3 ---
3 files changed, 1 insertion(+), 5 deletions(-)
diff --
From: Ian Romanick
v2: Update based on kernel interface / libdrm changes.
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/brw_context.h| 3 ++
src/mesa/drivers/dri/i965/brw_reset.c | 67 ++
3 fil
From: Ian Romanick
This allows drivers to determine whether a GPU reset has occured. It
should return non-zero status if a reset was observed by the specified
context. Another mechanism will be used to observe resets occuring in
other contexts in the share group.
Signed-off-by: Ian Romanick
-
From: Ian Romanick
If the application requests reset notifiction, connect up the reset
status query method and set gl_context::ResetStrategy.
v2: Update based on kernel interface / libdrm changes.
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_context.c | 26 +++
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/intel_screen.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
b/src/mesa/drivers/dri/i965/intel_screen.c
index eafafa2..6adcd34 100644
--- a/src/mesa/drivers/dri/i965
From: Ian Romanick
Soon some drivers will support a different set of flags than other
drivers. If some flags have to be filtered in the driver, we might as
well filter all of them in the driver.
The changes in nouveau use tabs because nouveau seems to have it's own
indentation rules.
Signed-of
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_context.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index 2bb52b6..9d5f348 100644
--- a/src/mesa/dr
From: Ian Romanick
No drivers advertise the DRI2 extension yet, so no driver should ever
see a value other than false for notify_reset.
The changes in nouveau use tabs because nouveau seems to have it's own
indentation rules.
Signed-off-by: Ian Romanick
---
src/gallium/state_trackers/dri/comm
From: Ian Romanick
These will be used to determine whether to signal a GPU reset after
another context in the share group has observed a reset.
Signed-off-by: Ian Romanick
---
src/mesa/main/context.c | 1 +
src/mesa/main/mtypes.h | 18 ++
2 files changed, 19 insertions(+)
di
From: Ian Romanick
Drivers still have to implement dd_function_table::GetGraphicsResetStatus.
Signed-off-by: Ian Romanick
---
src/mesa/main/getstring.c | 47 +++
1 file changed, 43 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/getstring.c
There are three parts this series:
- The first part cleans up some code that was speculatively added to
implement context reset noficition. This code didn't really work, so
it needed to be replaced.
- The second part wires up some necessary DRI2 bits so that drivers can
tell the loader tha
On Tue, Oct 29, 2013 at 7:06 PM, Niels Ole Salscheider
wrote:
> Hi Tom,
>
> this has been on my todo list for quite a while.
>
> Your patch looks good to me, but in my experience a block with approximately
> the same size for each dimension gives slightly better performance in many
> cases when co
On Tue, Oct 29, 2013 at 2:32 AM, Pohjolainen, Topi
wrote:
> On Mon, Oct 28, 2013 at 11:31:32AM -0700, Matt Turner wrote:
>> ---
>> src/mesa/drivers/dri/i965/brw_fs.cpp | 30 ++
>> src/mesa/drivers/dri/i965/brw_fs.h | 1 +
>> 2 files changed, 31 insertions(+)
>>
>> d
Commit aec20d66d9d13e0acd6a7199b63e1383e1e9900a
(automake: properly handle non-default expat installation),
assumed that up-to date distributions use a recent version
of expat that handles security vunerabilities CVE-2012-1147
and CVE-2012-1148. Seems like this is not always the case
and they prefe
On Tue, Oct 29, 2013 at 4:57 AM, Pohjolainen, Topi
wrote:
> On Mon, Oct 28, 2013 at 11:31:33AM -0700, Matt Turner wrote:
>> fs_visitor::try_replace_with_sel optimizes only if statements whose
>> "then" and "else" bodies contain a single MOV instruction. It also did
>> could not handle constant arg
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #7 from Emil Velikov ---
Created attachment 88329
--> https://bugs.freedesktop.org/attachment.cgi?id=88329&action=edit
handle expat versions pre 2.1
(In reply to comment #6)
> CVE-2012-1147 was determined not to be relevant on Ubun
On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry wrote:
> On 29 October 2013 15:59, Anuj Phogat wrote:
>
>>
>>
>>
>> On Mon, Oct 28, 2013 at 6:48 PM, Paul Berry wrote:
>>
>>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>>
- Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
We'd like to CSE some instructions, like CMP, that often have null
destinations. Instead of replacing them with MOVs to null, just don't
emit the MOV.
---
Updated version: Replaced two comments with Eric's suggested comment.
Left the comment about the basic block boundary in place. Let me know
if
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #6 from Bryce Harrington ---
CVE-2012-1147 was determined not to be relevant on Ubuntu:
http://people.canonical.com/~ubuntu-security/cve/2012/CVE-2012-1147.html
--
You are receiving this mail because:
You are the assignee for the b
Hi Tom,
this has been on my todo list for quite a while.
Your patch looks good to me, but in my experience a block with approximately
the same size for each dimension gives slightly better performance in many
cases when compared to one where one dimension is significantly larger.
Maybe you coul
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #5 from Bryce Harrington ---
Created attachment 88328
--> https://bugs.freedesktop.org/attachment.cgi?id=88328&action=edit
Hardcode EXPAT in configure.ac
expat 2.0 does not provide the pkgconfig file.
Hardcoding EXPAT as suggested
On Mon, Oct 28, 2013 at 11:08 PM, Eric Anholt wrote:
> Matt Turner writes:
>
>> Since compare instructions write the flag register, they should not be
>> considered dead even if their destination is never read. Instead of
>> removing them if found to be dead, set their destination to null to free
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #4 from Emil Velikov ---
(In reply to comment #3)
> expat-2.1.0 was released March 24 2012. That's kind of a long time for it to
> not be in your distribution.
>
> I'm personally okay with requiring 2.1.0, but I suppose people want n
On Tue, Oct 29, 2013 at 2:26 PM, Matt Turner wrote:
> But hopefully your patch just avoids this problem entirely. I'll
> provide an update later.
7 shader helped, 4 shaders hurt by going from my series and your
patch, to my series, your patch, and disabling the original
SEL-generating function fo
From: Marek Olšák
The type-2 NOPs are said to be unstable. It doesn't make a difference here.
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
b/src/gallium/winsys/radeon/
From: Marek Olšák
This doesn't make any difference on Bonaire, but it might help on Hawaii.
---
src/gallium/drivers/radeonsi/si_state.c | 1 +
src/gallium/drivers/radeonsi/si_state.h | 1 +
src/gallium/drivers/radeonsi/si_state_draw.c | 8 ++--
3 files changed, 8 insertions(+), 2 d
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #3 from Matt Turner ---
expat-2.1.0 was released March 24 2012. That's kind of a long time for it to
not be in your distribution.
I'm personally okay with requiring 2.1.0, but I suppose people want new Mesa on
old things.
I might su
This can deal with all the 15 32-bit untyped atomic operations the
hardware supports, but only INC and PREDEC are going to be exposed
through the API for now.
v2: Represent atomics as GLSL intrinsics. Add support for variably
indexed atomic counter arrays.
---
src/mesa/drivers/dri/i965/brw_v
The latency information has been obtained empirically from
measurements taken on Haswell and Ivy Bridge.
---
.../drivers/dri/i965/brw_schedule_instructions.cpp | 41 ++
1 file changed, 41 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
b/src
This can deal with all the 15 32-bit untyped atomic operations the
hardware supports, but only INC and PREDEC are going to be exposed
through the API for now.
v2: Represent atomics as GLSL intrinsics. Add support for variably
indexed atomic counter arrays. Fix interaction with fragment
d
v2: Add comments on the purpose of the auxiliary data structures.
Check for atomic counter overlaps. Use the contains_atomic()
convenience method. Add static assert with the number of expected
shader stages.
v3: Don't resize atomic arrays.
---
src/glsl/Makefile.sources | 1 +
src/
I merged most of my atomic-counters branch a few hours ago, however
some of the patches haven't received much attention and I don't feel
confident enough with pushing them yet, even though they have been
sitting on the mailing list for more than six weeks.
I'm resending only those patches that I'd
v2: Mark atomic counters as read-only variables. Move offset overlap
code to the linker. Use the contains_atomic() convenience method.
v3: Use pointer to integer instead of non-const reference. Add
comment so we remember to add a spec quotation from the next GLSL
release once the iss
This patch fixes the three dead code elimination passes and the
VEC4/FS instruction scheduling passes so they leave instructions with
side effects alone.
At some point it might be interesting to have the instruction
scheduler calculate the exact memory dependencies between atomic ops,
but they're
On 29 October 2013 15:59, Anuj Phogat wrote:
>
>
>
> On Mon, Oct 28, 2013 at 6:48 PM, Paul Berry wrote:
>
>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>
>>> - Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
>>> GEN7_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
Crap. Two patches were mistakenly squashed together resulting in this.
I'll resend.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #2 from Bryce Harrington ---
Reverting patch aec20d66 enables it to pass the configure check.
commit aec20d66d9d13e0acd6a7199b63e1383e1e9900a
Author: Emil Velikov
Date: Tue Oct 29 21:14:41 2013 +
automake: properly handle
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #1 from Bryce Harrington ---
Same error on ubuntu 12.04
$ apt-cache policy expat
expat:
Installed: 2.0.1-7.2ubuntu1.1
Candidate: 2.0.1-7.2ubuntu1.1
Looks like expat.pc was added in expat 2.1. 2.0 doesn't install it, so mesa's
p
On Mon, Oct 28, 2013 at 6:48 PM, Paul Berry wrote:
> On 25 October 2013 16:45, Anuj Phogat wrote:
>
>> - Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
>> GEN7_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
>> - Only enable one of GEN7_WM_8_DISPATCH_ENABLE or
>> GE
https://bugs.freedesktop.org/show_bug.cgi?id=71022
Priority: medium
Bug ID: 71022
Keywords: regression
CC: emil.l.veli...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: configure: error: Expat required for DRI.
On 28/09/13 15:46, Emil Velikov wrote:
> Slight extension over the previous two patches, removing unused variables
> and headers. Last two patches do not strictly below in this series, as
> they provide more consistent automake/configure usage of pkg-config
> (PKG_CHECK_MODULES).
>
> Please commen
On Tue, Oct 29, 2013 at 12:36 PM, Eric Anholt wrote:
> Matt Turner writes:
>
>> Helps a lot of Steam games.
>>
>> total instructions in shared programs: 1287432 -> 1287130 (-0.02%)
>> instructions in affected programs: 21447 -> 21145 (-1.41%)
>
> 13-15 are also:
>
> Reviewed-by: Eric Anholt
From: Tom Stellard
DRM_IOCTL_VERSION is not currently a legal ioctl for render nodes, so we
need to hard code the drm version when drmGetVersion() returns NULL.
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 36 ---
1 file changed, 25 insertions(+), 11 deletions(-)
From: Tom Stellard
You can use the --enable-pipe-loader-render-nodes configure flag to
make the pipe-loader use render nodes for talking with the device.
---
configure.ac | 6 ++
src/gallium/auxiliary/pipe-loader/Makefile.am | 5 ++
.../auxiliary/pipe
This is a resend of the series after updates for Paul's review feedback.
This time I've done a full piglit run with spilling forced on and no
regressions were found. It can also be found at gen7-scratch-read of my
tree.
___
mesa-dev mailing list
mesa-de
We were clearing the reg_offset before trying to use it. Oops. Fixes
glsl-fs-texture2drect with the reg spilling debug enabled.
Reviewed-by: Paul Berry
---
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drive
I believe this will never happen in SIMD8 mode, but it could for SIMD16
when we fix it.
---
src/mesa/drivers/dri/i965/brw_fs.h| 1 +
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 23 +++
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 ++
3 files c
Now that reg spilling generates new vgrfs, we were looping forever if you
ever turned it on.
Instead, move the debug code into the register allocator right near where
we'd be doing spilling anyway, which should more accurately reflect how
register spilling occurs in the wild.
Reviewed-by: Paul Be
I'm going to need to reuse this for fixing register spilling on SIMD16.
Note that BRW_MAX_MRF is 16, which is the same as BRW_MAX_GRF -
GEN7_MRF_HACK_START.
---
src/mesa/drivers/dri/i965/brw_fs.h| 1 +
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 33 ---
Things blew up when I enabled the debug register spill code without
disabling 16-wide, so I decided to just fix 16-wide spilling.
We still don't generate 16-wide when register spilling happens as part of
allocation (since we expect it to be slower), but now we can experiment
with allowing it in so
This avoids a lot of message setup we had to do otherwise. Improves
GLB2.7 performance with register spilling force enabled by 1.6442% +/-
0.553218% (n=4).
v2: Use BRW_PREDICATE_NONE, improve a comment (by Paul).
Reviewed-by: Paul Berry
---
src/mesa/drivers/dri/i965/brw_defines.h|
I'm going to be introducing gen7 variants, and the previous naming was
going to get confusing.
Reviewed-by: Paul Berry
---
src/mesa/drivers/dri/i965/brw_defines.h | 7 +++
src/mesa/drivers/dri/i965/brw_fs.cpp| 4 ++--
src/mesa/drivers/dri/i965/brw_fs.h
This hasn't been true since SIMD16 mode was added.
---
src/mesa/drivers/dri/i965/brw_fs.h | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h
b/src/mesa/drivers/dri/i965/brw_fs.h
index 5b78313..5f331e1 100644
--- a/src/mesa/drivers/dri/i9
On 10/29/2013 02:10 PM, Rob Clark wrote:
From: Rob Clark
A convenient front end to indices generate/translate code, for emulating
primitives which are not supported natively by the driver.
This handles saving/restoring index buffer state, etc.
Signed-off-by: Rob Clark
Reviewed-by: Brian Paul
Josh Klint writes:
> Hi,
>
> As I understand it, Mesa is in charge of providing OpenGL drivers for
> Intel graphics hardware running on Linux. I'm in the process of
> porting our game engine over to Linux, and have been pleasantly
> surprised so far that the OpenGL 4 drivers for Nvidia and ATI o
---
src/mesa/main/performance_monitor.c |6 +++---
src/mesa/main/uniforms.c|2 +-
src/mesa/main/vdpau.c |6 --
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/performance_monitor.c
b/src/mesa/main/performance_monitor.c
index 8
---
src/mesa/vbo/vbo_attrib_tmp.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/vbo/vbo_attrib_tmp.h b/src/mesa/vbo/vbo_attrib_tmp.h
index 02c283d..bbc0205 100644
--- a/src/mesa/vbo/vbo_attrib_tmp.h
+++ b/src/mesa/vbo/vbo_attrib_tmp.h
@@ -140,7 +140,7 @@ stati
Fixes "warning: cast from pointer to integer of different size" for
64-bit builds.
---
src/mesa/state_tracker/st_draw.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_draw.h b/src/mesa/state_tracker/st_draw.h
index 3313fc8..394473b 100644
--- a/src
---
src/glsl/glsl_types.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/glsl_types.h b/src/glsl/glsl_types.h
index fdb1f3a..96eee5e 100644
--- a/src/glsl/glsl_types.h
+++ b/src/glsl/glsl_types.h
@@ -459,7 +459,7 @@ struct glsl_type {
*/
bool contains_atom
---
src/mesa/drivers/common/meta.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 798efa6..aa50dde 100644
--- a/src/mesa/drivers/common/meta.c
+++ b/src/mesa/drivers/common/m
---
src/mesa/swrast/s_texfetch_tmp.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/swrast/s_texfetch_tmp.h b/src/mesa/swrast/s_texfetch_tmp.h
index 7a68753..e2521b5 100644
--- a/src/mesa/swrast/s_texfetch_tmp.h
+++ b/src/mesa/swrast/s_texfetch_tmp.h
@@ -1537,7
From: Rob Clark
This patchset (compared to RFC I sent previously) changes u_primconvert
to just be a front-end to the u_indices stuff. It handles binding/
restoring new index buffer state, etc. So driver using it just has
to put this at the top of their pipe->draw_vbo():
if (prim_needs_emula
From: Rob Clark
Add 'start' parameter to generator/translator.
Signed-off-by: Rob Clark
Reviewed-by: Brian Paul
---
v1: original
v2: add comments
src/gallium/auxiliary/indices/u_indices.c | 6 --
src/gallium/auxiliary/indices/u_indices.h | 21 -
src
From: Rob Clark
Use u_primconvert to convert unsupported primitives into supported
primitive plus index buffer.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/a2xx/fd2_context.c | 24 ++-
src/gallium/drivers/freedreno/a3xx/fd3_context.c | 12 +-
src/galliu
From: Rob Clark
A convenient front end to indices generate/translate code, for emulating
primitives which are not supported natively by the driver.
This handles saving/restoring index buffer state, etc.
Signed-off-by: Rob Clark
Reviewed-by: Brian Paul
---
v1: original
v2: re-indent, fix initi
Hi,
As I understand it, Mesa is in charge of providing OpenGL drivers for Intel
graphics hardware running on Linux. I'm in the process of porting our game
engine over to Linux, and have been pleasantly surprised so far that the OpenGL
4 drivers for Nvidia and ATI on Linux work perfectly:
http:
Matt Turner writes:
> Helps a lot of Steam games.
>
> total instructions in shared programs: 1287432 -> 1287130 (-0.02%)
> instructions in affected programs: 21447 -> 21145 (-1.41%)
13-15 are also:
Reviewed-by: Eric Anholt
I'm spending a while thinking about the if->sel passes. Having 2
Tom Stellard schrieb am 29.10.2013 17:48:
> On Sun, Oct 27, 2013 at 07:36:07PM +0100, Kai Wasserbäch wrote:
>> Fixes a boat load of Piglit tests for me, which crashed like fdo#70913
>> before.
>>
>> Thanks to Michel Dänzer for the tip.
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70
On 29 October 2013 12:04, Eric Anholt wrote:
> Paul Berry writes:
>
> > On 29 October 2013 11:20, Eric Anholt wrote:
> >
> >> Paul Berry writes:
> >>
> >> > On 27 October 2013 12:58, Marek Olšák wrote:
> >> >
> >> >> On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry >
> >> >> wrote:
> >> >> > On 2
Paul Berry writes:
> On 29 October 2013 11:20, Eric Anholt wrote:
>
>> Paul Berry writes:
>>
>> > On 27 October 2013 12:58, Marek Olšák wrote:
>> >
>> >> On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry
>> >> wrote:
>> >> > On 20 October 2013 09:20, Marek Olšák wrote:
>> >> >>
>> >> >> From: Mare
On Tue, Oct 29, 2013 at 12:36 AM, Eric Anholt wrote:
> Anuj Phogat writes:
>
> > V2:
> >- Update comments.
> >- Make changes to support simd16 mode.
> >- Add compute_pos_offset variable in brw_wm_prog_key.
> >- Add variable uses_omask in brw_wm_prog_data.
> >
> > Signed-off-by: A
Paul Berry writes:
> On 21 October 2013 17:48, Eric Anholt wrote:
>
>> Things blew up when I enabled the debug register spill code without
>> disabling 16-wide, so I decided to just fix 16-wide spilling.
>>
>> We still don't generate 16-wide when register spilling happens as part of
>> allocatio
On 10/28/2013 04:29 PM, Emil Velikov wrote:
On 28/10/13 08:07, Adrian Negreanu wrote:
From: Adrian Negreanu
Fixes linker error:
ld: .../libmesa_dri_common_intermediates/libmesa_dri_common.a(dri_util.o):
in function globalDriverAPI:dri_util.c(.data.rel+0x0): error: undefined
reference
On 29 October 2013 11:20, Eric Anholt wrote:
> Paul Berry writes:
>
> > On 27 October 2013 12:58, Marek Olšák wrote:
> >
> >> On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry
> >> wrote:
> >> > On 20 October 2013 09:20, Marek Olšák wrote:
> >> >>
> >> >> From: Marek Olšák
> >> >>
> >> >> This avo
From: Tom Stellard
This results in huge performance improvements for applications like
pyrit, which depend on the implementation to determine the optimal
work group size.
---
src/gallium/state_trackers/clover/api/kernel.cpp | 9 -
src/gallium/state_trackers/clover/core/device.cpp | 19
From: Tom Stellard
This results in huge performance improvements for applications like
pyrit, which depend on the implementation to determine the optimal
work group size.
---
src/gallium/state_trackers/clover/api/kernel.cpp | 9 -
src/gallium/state_trackers/clover/core/device.cpp | 19
Paul Berry writes:
> On 27 October 2013 12:58, Marek Olšák wrote:
>
>> On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry
>> wrote:
>> > On 20 October 2013 09:20, Marek Olšák wrote:
>> >>
>> >> From: Marek Olšák
>> >>
>> >> This avoids a defect in lower_output_reads.
>> >>
>> >> The problem is lower
On Mon, Oct 28, 2013 at 7:00 PM, Paul Berry wrote:
> On 28 October 2013 18:14, Paul Berry wrote:
>
>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>
>>> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
>>> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
>>> -
On 27 October 2013 14:59, Ian Romanick wrote:
> From: Ian Romanick
>
> This will simplify the addition of layout(location) qualifiers for
> separate shader objects. This was validated with new piglit tests
> arb_explicit_attrib_location/1.30/compiler/not-enabled-01.vert and
> arb_explicit_attri
On 27 October 2013 14:59, Ian Romanick wrote:
> From: Ian Romanick
>
> Future patches will add some extra code to this path, and some of that
> code will want to exit from the explicit location code early.
>
> Signed-off-by: Ian Romanick
> ---
> src/glsl/ast_to_hir.cpp | 159
>
On 27 October 2013 14:59, Ian Romanick wrote:
> From: Ian Romanick
>
> I made this a function (instead of a method of ir_variable) because it
> made the change set smaller, and I expect that there will be an overload
> that takes an ir_var_mode enum. Having both functions used the same way
> se
On 27 October 2013 14:59, Ian Romanick wrote:
> From: Gregory Hainaut
>
> The return value has been unused since commit d348b0c. This was
> originally included in another patch, but it was split out by Ian
> Romanick.
>
> Reviewed-by: Ian Romanick
> Cc: Eric Anholt
> ---
> src/mesa/main/shad
On Sat, Oct 26, 2013 at 07:54:27PM +0200, David Heidelberger wrote:
> From f188c10862f30205a7838a2e5f09d3e3a9b18f59 Mon Sep 17 00:00:00 2001
> From: David Heidelberger
> Date: Fri, 25 Oct 2013 18:07:05 +0200
> Subject: [PATCH v2] clover: mark gcc 4.7.0 as broken
>
> It certainly not compile on 4.
From: Tom Stellard
The ICD loader should be responsible for installing headers.
---
src/gallium/state_trackers/clover/Makefile.am | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/Makefile.am
b/src/gallium/state_tracker
On Sun, Oct 27, 2013 at 07:36:07PM +0100, Kai Wasserbäch wrote:
> Fixes a boat load of Piglit tests for me, which crashed like fdo#70913
> before.
>
> Thanks to Michel Dänzer for the tip.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70913
> Signed-off-by: Kai Wasserbäch
This looks
On Fri, Oct 25, 2013 at 05:46:38PM -0500, Aaron Watry wrote:
> Reviewed-by: Aaron Watry
>
> I have tested this on a Radeon 5400 (Cedar), and I just sent a few
> generated tests to the piglit list.
>
> --Aaron
Thanks for testing, I have committed this patch.
-Tom
>
> On Wed, Oct 23, 2013 at 6
On 27 October 2013 12:58, Marek Olšák wrote:
> On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry
> wrote:
> > On 20 October 2013 09:20, Marek Olšák wrote:
> >>
> >> From: Marek Olšák
> >>
> >> This avoids a defect in lower_output_reads.
> >>
> >> The problem is lower_output_reads treats the gl_FragD
On 26 October 2013 00:13, Vinson Lee wrote:
> Fixes "Uninitialized pointer field" defect reported by Coverity.
>
> Signed-off-by: Vinson Lee
>
Reviewed-by: Paul Berry
> ---
> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 7 ++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> dif
On 10/28/2013 06:01 PM, Alexander von Gluck IV wrote:
* Instead of assuming the displaytarget is the same
stride / colorspace as the destination, lets
actually check the source bitmap.
* Fixes random stride issues in rendering
---
src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp | 20 ++
On 29 October 2013 00:53, Tapani wrote:
> On 10/28/2013 10:09 PM, Paul Berry wrote:
>
> +static uint32_t _unique_id(ir_variable *var)
>> +{
>> + char buffer[256];
>> + _mesa_snprintf(buffer, 256, "%s_%p", var->name, var);
>> + return _mesa_str_checksum(buffer);
>>
>
> Two problems:
>
> 1.
On 29 October 2013 00:28, Eric Anholt wrote:
> Paul Berry writes:
>
> > On 21 October 2013 11:20, Eric Anholt wrote:
> >
> >> Orbital Explorer was generating a 4000 instruction geometry shader,
> which
> >> was taking 275 trips through dead code elimination and register
> >> coalescing, each of
On Mon, Oct 28, 2013 at 11:31:33AM -0700, Matt Turner wrote:
> fs_visitor::try_replace_with_sel optimizes only if statements whose
> "then" and "else" bodies contain a single MOV instruction. It also did
> could not handle constant arguments, since they cause an extra MOV
> immediate to be generate
1 - 100 of 105 matches
Mail list logo
