the best solution is. I'm mainly trying to raise awareness of the
issues.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev
without opengl and openles2, we want to know
whether mesa-demos opengles2 folder will compile without any issues?
or any code changes required to compile the es2 demos? for X11.
None of this should matter for the compilation of the demos.
--
Earthling Michel Dänzer
'? :)
Slightly more seriously, I have /dev/std{out,err} files for the purpose
of this patch, but I suppose those probably aren't available everywhere
this code can run...
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian
for GL_INT. :)
- for 8_8 formats, instead of littleEndian !swapBytes,
use littleEndian != swapBytes to include big endian hosts
Based on suggestions from Michel Dänzer.
Thanks for tackling this!
case MESA_FORMAT_DUDV8:
return (format == GL_DU8DV8_ATI || format == GL_DUDV_ATI
From: Michel Dänzer michel.daen...@amd.com
GLX uses mapi/glapi/libglapi.la, which is only built for OpenGL.
NOTE: This is a candidate for the stable branches.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59364
Tested-by: Tom Stellard thomas.stell...@amd.com
Signed-off-by: Michel
On Don, 2013-01-31 at 18:03 +0100, Marek Olšák wrote:
On Thu, Jan 31, 2013 at 5:39 PM, Michel Dänzer mic...@daenzer.net wrote:
I'm afraid these are not quite correct yet. I'm not 100% sure after
looking at the red book and the GL 3.x spec — this is head spinning
material, isn't
;
- else
- return 32;
+ return 32;
case PIPE_SHADER_CAP_MAX_TEMPS:
return 256; /* Max native temporaries. */
case PIPE_SHADER_CAP_MAX_ADDRS:
Reviewed-by: Michel Dänzer michel.daen...@amd.com
P.S. The subject of patch 1 should say
From: Michel Dänzer michel.daen...@amd.com
Handle vectors of 1 to 16 integers.
Change the intrinsic names to prevent the wrong one from being selected at
runtime due to the overloading.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstrInfo.td|2 +-
lib
From: Michel Dänzer michel.daen...@amd.com
Use sub0-15 everywhere.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/AMDGPUInstructions.td |4 +--
lib/Target/R600/AMDGPURegisterInfo.cpp | 32 ++---
lib/Target/R600/AMDGPURegisterInfo.td |7
This series and the corresponding radeonsi series get shadow samplers and
texture arrays mostly working and fix a bunch of other piglit failures,
gaining just shy of 100 piglits.
[PATCH 1/4] R600: Consolidate sub register indices.
[PATCH 2/4] R600/SI: Add basic support for more integer vector
From: Michel Dänzer michel.daen...@amd.com
v1i32, v2i32, v8i32 and v16i32.
Only add VGPR register classes for integer vector types, to avoid attempts
copying from VGPR to SGPR registers, which is not possible.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/R600Instructions.td | 16 -
lib/Target/R600/SIInstructions.td | 42 ---
2 files changed, 54 insertions(+), 4 deletions(-)
diff
From: Michel Dänzer michel.daen...@amd.com
In particular, the LOD bias and depth comparison values are packed before the
'normal' texture coordinates, and the array slice and LOD values are appended.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi
From: Michel Dänzer michel.daen...@amd.com
Fix up intrinsic names, and bitcast texture address parameters to integers.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeon/radeon_llvm.h |2 +-
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 50
From: Michel Dänzer michel.daen...@amd.com
28/30 piglit tests pass.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
b/src
From: Michel Dänzer michel.daen...@amd.com
The hardware can't do it, and these were causing warnings in some piglit tests.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/si_state.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/src/gallium/drivers
Tom provided without causing any regression.
So please test again, and apply if it now works correctly.
Git complained about the patches introducing trailing whitespace, but
the series is
Tested-by: Michel Dänzer michel.daen...@amd.com
--
Earthling Michel Dänzer
;
/* SWTCL-only features. */
Thanks for adding the cap to r300g, but what about r600g and radeonsi?
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev
From: Michel Dänzer michel.daen...@amd.com
22 more little piglits with radeonsi.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target
(HI32 imm:$imm)), high)
+;
You need to rebase this onto my changes Tom just pushed to his master
branch, using 'sub0' instead of 'low' and 'sub1' instead of 'high'.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian
From: Michel Dänzer michel.daen...@amd.com
20 more little piglits with radeonsi.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td |4
1 file changed, 4 insertions(+)
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600
? :)
But I'm happy to let those that will do this work decide.
Likewise.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev
On Mon, 2013-02-04 at 20:17 +0100, Michel Dänzer wrote:
[...] how about something like this:
Define the packing to be in the host byte order. However, do not define
array formats as packed values (which makes little sense e.g. for
*32*32*32*32 anyway) but really just as arrays. 16- or 32
*/
case PIPE_SHADER_CAP_MAX_CONSTS:
- return 64;
+ return 4096;
Where does the value 4096 come from? Maybe add a comment.
Other than that, this series is
Reviewed-by: Michel Dänzer michel.daen...@amd.com
Also, this series and the LLVM series are
Tested-by: Michel
From: Michel Dänzer michel.daen...@amd.com
The hardware can't do it.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/si_state.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium
From: Michel Dänzer michel.daen...@amd.com
18 more little piglits.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/si_state.c | 27 ++-
1 file changed, 18 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi
From: Michel Dänzer michel.daen...@amd.com
Allows nexuiz to run with radeonsi.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstrInfo.cpp | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstrInfo.cpp b/lib/Target
From: Michel Dänzer michel.daen...@amd.com
Append the overloaded vector type used for passing in the addressing
parameters.
Without this, LLVM uses the same function signature for all those types,
which cannot work.
Fixes problems e.g. with FlightGear and Red Eclipse.
Signed-off-by: Michel
From: Michel Dänzer michel.daen...@amd.com
Also, add assertions to stress that render targets don't support scaled
formats.
20 more little piglits.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
Updated, more extensive patch for 2 extra little piglits.
src/gallium/drivers/radeonsi
From: Michel Dänzer michel.daen...@amd.com
The modifiers don't seem to have any effect with V_MOV_B32, supposedly it's
meant to just move bits untouched.
Fixes 46 piglit tests with radeonsi, though unfortunately 11 of those had
just regressed because they started using the clamp modifier
On Don, 2013-02-07 at 21:34 -0800, Ian Romanick wrote:
On 01/31/2013 08:50 AM, Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
GLX uses mapi/glapi/libglapi.la, which is only built for OpenGL.
NOTE: This is a candidate for the stable branches.
Bugzilla: https
for librt because 'there is no librt'. :)
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
by either of these two files
and the radeon driver still works on Alpha, so the definitions are dead
and not needed.
Reviewed-by: Michel Dänzer michel.daen...@amd.com
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian
sense for (fadd (fmul)))
while mine used V_MAD_F32. Same problem using V_FMA_F32 as well. Still
not sure why it breaks. :(
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
but only the final sum?
Patch 1 looks nice and is
Reviewed-by: Michel Dänzer michel.daen...@amd.com
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
this for the Mesa stable branch...
Tom / Christian, what's your opinion on this?
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
On Mon, 2013-02-11 at 16:12 +0100, Tom Stellard wrote:
On Mon, Feb 11, 2013 at 03:35:44PM +0100, Michel Dänzer wrote:
On Son, 2013-02-10 at 19:38 +0100, Vincent Lejeune wrote:
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index a09f243
From: Michel Dänzer michel.daen...@amd.com
'R600/SI: Use proper instructions for array/shadow samplers.' removed two
cases from TEX_SHADOW. Vincent Lejeune reported on IRC that this broke some
shadow array piglit tests with the r600g driver. Reinstating the removed
cases should fix
.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
spelling would be 'intrust' or 'entrust', but I think the
proper verb here is simply 'trust'.
Same comments apply to r600_state.c.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
test/CodeGen/R600/llvm.AMDGPU.tex.ll | 42 +
test/CodeGen/R600/llvm.SI.sample.ll | 71
2 files changed, 113 insertions(+)
create mode
On Die, 2013-02-12 at 15:26 +0100, Tom Stellard wrote:
On Tue, Feb 12, 2013 at 10:24:24AM +0100, Michel Dänzer wrote:
On Mon, 2013-02-11 at 19:39 +0100, Vincent Lejeune wrote:
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 24
1 file changed, 24 insertions
From: Michel Dänzer michel.daen...@amd.com
8 more little piglits.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 29 ++
1 file changed, 29 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
());
Imm.F = Op.getFPImm();
}
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
value.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman
On Mit, 2013-02-13 at 10:16 +0100, Christian König wrote:
Am 12.02.2013 21:49, schrieb Michel Dänzer:
On Die, 2013-02-12 at 18:13 +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
Mark all the operands that can also have an immediate.
Signed-off
From: Michel Dänzer michel.daen...@amd.com
The important fix is that the constant interpolation value is stored in the
parameter slot P0, which is encoded as 2.
In addition, pass the parameter slot as an operand to V_INTERP_MOV_F32
instead of hardcoding it there, and add a special register class
From: Michel Dänzer michel.daen...@amd.com
Requires corresponding LLVM R600 backend fix to work correctly, but even
without that it doesn't hang anymore.
13 more little piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium
From: Michel Dänzer michel.daen...@amd.com
It's the reciprocal of the register value.
Fixes piglit fragcoord_w and glsl-fs-fragcoord-zw-perspective.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi
From: Michel Dänzer michel.daen...@amd.com
The important fix is that the constant interpolation value is stored in the
parameter slot P0, which is encoded as 2.
In addition, pass the parameter slot as an operand to V_INTERP_MOV_F32
instead of hardcoding it there, and add a special register class
such as Vincent's
MAD changes or switching to the Source scheduler from breaking stuff.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote:
Am 13.02.2013 18:11, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 11:34 -0500, Tom Stellard wrote:
There's just the one cleanup on patch 10 that you mentioned, but
otherwise the series looks good to me. Should we mark all
From: Michel Dänzer michel.daen...@amd.com
9 more little piglits with radeonsi.
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
From: Michel Dänzer michel.daen...@amd.com
16 more little piglits with radeonsi.
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td | 5 +
1 file changed, 5 insertions(+)
diff --git a/lib/Target
From: Michel Dänzer michel.daen...@amd.com
24 more little piglits with radeonsi.
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td | 4
1 file changed, 4 insertions(+)
diff --git a/lib/Target
0x0080e479
Stack dump:
0. Program arguments: build-trunk/Debug+Asserts/bin/llc -march=r600
-mcpu=SI
1. Running pass 'Function Pass Manager' on module 'stdin'.
2. Running pass 'SI annotate control flow' on function '@test'
--
Earthling Michel Dänzer
On Don, 2013-02-14 at 11:32 +0100, Christian König wrote:
Am 13.02.2013 18:22, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote:
Am 13.02.2013 18:11, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 11:34 -0500, Tom Stellard wrote:
There's just the one cleanup
.addImm(0) // OMOD
For me this resulted in encoding 0 directly (because these operands are
not declared as VSrc_32), i.e. VGPR0, which is no good here.
addImm(0x80) for inline constant 0 worked for me.
--
Earthling Michel Dänzer | http://www.amd.com
Libre
On Don, 2013-02-14 at 14:35 +0100, Michel Dänzer wrote:
On Don, 2013-02-14 at 11:43 +0100, Christian König wrote:
+// Only one literal allowed
+break;
The same literal could be used for several operands though. :)
Basically, it seems like we're pretty close to allowing VSrc
On Don, 2013-02-14 at 15:14 +0100, Michel Dänzer wrote:
On Don, 2013-02-14 at 14:35 +0100, Michel Dänzer wrote:
On Don, 2013-02-14 at 11:43 +0100, Christian König wrote:
+// Only one literal allowed
+break;
The same literal could be used for several operands though
On Don, 2013-02-14 at 16:20 +0100, Christian König wrote:
Am 14.02.2013 15:14, schrieb Michel Dänzer:
On Don, 2013-02-14 at 14:35 +0100, Michel Dänzer wrote:
On Don, 2013-02-14 at 11:43 +0100, Christian König wrote:
+// Only one literal allowed
+break;
The same literal could
On Mit, 2013-02-13 at 17:51 +0100, Christian König wrote:
Am 13.02.2013 17:07, schrieb Michel Dänzer:
From: Michel Dänzer michel.daen...@amd.com
The important fix is that the constant interpolation value is stored in the
parameter slot P0, which is encoded as 2.
In addition, pass
From: Michel Dänzer michel.daen...@amd.com
The important fix is that the constant interpolation value is stored in the
parameter slot P0, which is encoded as 2.
In addition, drop the SI_INTERP_CONST pseudo instruction, pass the parameter
slot as an operand to V_INTERP_MOV_F32 instead
at least some of
the lessons learned from your branch to the LLVM backend as well.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing
From: Michel Dänzer michel.daen...@amd.com
Based on r600g commit 2b9659c9e627ad03160899b8be04f96307d098eb .
Fixes crashes with 4 piglit tests which are now hitting these formats.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium
From: Michel Dänzer michel.daen...@amd.com
7 more little dwarves^W piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/r600_texture.c | 3 +--
src/gallium/drivers/radeonsi/si_state.c | 14
From: Michel Dänzer michel.daen...@amd.com
4 more little piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/si_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium
On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote:
On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer mic...@daenzer.net wrote:
Really, what I don't understand is why r600g doesn't seem affected by
this... at least on my RS880 it's passing the piglit tests this change
fixes with radeonsi
, could you add the operands
to them?
let SRC2 = 0x80;
Hmm, we're scattering quite a few of these magic 0x80 around, would be
nice to make those more self-documenting somehow...
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast
On Die, 2013-02-19 at 15:48 +0100, Marek Olšák wrote:
On Tue, Feb 19, 2013 at 3:28 PM, Michel Dänzer mic...@daenzer.net wrote:
On Die, 2013-02-19 at 14:04 +0100, Marek Olšák wrote:
On Tue, Feb 19, 2013 at 11:02 AM, Michel Dänzer mic...@daenzer.net wrote:
Really, what I don't understand
From: Marek Olšák mar...@gmail.com
[ Cherry-picked from r600g commit b278aba42310e8fa30f2408b9dcd58dbb4901724 ]
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/r600_texture.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src
These together get us 11 more little piglits with Marek's
glTex(Sub)Image improvements in st/mesa.
[PATCH 1/3] radeonsi: use u_box_origin_2d helper function
[PATCH 2/3] radeonsi: add assertions to prevent creation of invalid
[PATCH 3/3] radeonsi: implement 3D transfers
From: Marek Olšák mar...@gmail.com
[ Cherry-picked from r600g commit ef11ed61a0414d0405c3faf7f48fa3f1d083f82e ]
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/r600_blit.c | 15 ---
src/gallium/drivers/radeonsi/r600_texture.c | 2 ++
src
From: Marek Olšák mar...@gmail.com
That means we can map and read multiple slices with one transfer_map call.
[ Cherry-picked from r600g commit 1aebb6911e9aa1bd8900868b58d1750ca83a20c7 ]
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/r600_texture.c | 49
(type=%u)
%d\n,
sel-type, r);
sel-current = NULL;
+ FREE(shader);
return r;
}
Reviewed-by: Michel Dänzer michel.daen...@amd.com
Can you fix the same problem in r600g while you're
From: Michel Dänzer michel.daen...@amd.com
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
Not sure why the lack of this didn't seem to cause problems in my testing
last week...
lib/Target/R600/SIInstructions.td | 3
From: Michel Dänzer michel.daen...@amd.com
Only compile tested, but should fix at least some piglit fbo-blending tests.
NOTE: This is a candidate for the stable branches.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/r600/evergreen_state.c | 13 +
1
From: Michel Dänzer michel.daen...@amd.com
11 more little piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
v2: Marek pointed out there actually is a hardware bit for this as of
Cayman. :)
src/gallium/drivers/radeonsi
On Don, 2013-02-14 at 11:32 +0100, Christian König wrote:
Am 13.02.2013 18:22, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote:
Am 13.02.2013 18:11, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 11:34 -0500, Tom Stellard wrote:
There's just the one cleanup
*/)
+;
Is there any advantage to doing it like this as opposed to using
V_MUL_F32_e32 with the corresponding inline constant source operands? I
guess it could work for SGPRs as the source as well, if the patterns
used VSrc instead of VReg, but otherwise MUL would result in more
compact code.
--
Earthling Michel
for the asm string
changes.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
On Don, 2013-02-21 at 09:44 +0100, Christian König wrote:
Am 21.02.2013 09:26, schrieb Michel Dänzer:
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index 833e6cd..7c157e9 100644
On Mit, 2013-02-20 at 17:56 +0100, Michel Dänzer wrote:
On Don, 2013-02-14 at 11:32 +0100, Christian König wrote:
Am 13.02.2013 18:22, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote:
Am 13.02.2013 18:11, schrieb Michel Dänzer:
On Mit, 2013-02-13 at 11
On Don, 2013-02-21 at 10:38 +0100, Christian König wrote:
Am 21.02.2013 09:39, schrieb Michel Dänzer:
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index 700b8f8..866c7cb 100644
--- a/lib
assignment looks superfluous now.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 24
lib/Target/R600/SIInstrInfo.td| 22 ++
lib/Target/R600/SIInstructions.td | 4
3 files changed, 50
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstrInfo.cpp | 28 +++-
1 file changed, 27 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstrInfo.cpp b/lib/Target/R600/SIInstrInfo.cpp
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/SIInstructions.td | 7 ++-
lib/Target/R600/SIIntrinsics.td | 1 +
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstructions.td
b/lib
From: Michel Dänzer michel.daen...@amd.com
Use LDS for calculating the deltas between neighbouring pixels. Not sure the
sign of the delta is correct for both dimensions (the sign doesn't seem to
matter for the relevant piglit tests), but it'll be easy to fix that up if
not.
Signed-off-by: Michel
From: Michel Dänzer michel.daen...@amd.com
The number of coordinates to pack will be useful for other address parameters
as well.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 54 +++---
1 file changed, 32 insertions
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 23 +--
1 file changed, 21 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src
From: Michel Dänzer michel.daen...@amd.com
15 more little piglits.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 34 ++
src/gallium/drivers/radeonsi/radeonsi_shader.h | 1 +
src/gallium/drivers/radeonsi
for any source
operand, and it automagically uses an inline constant, a literal or a
temporary register as appropriate. It sounded like Christian is getting
close to that.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast
On Fre, 2013-02-22 at 09:05 -0800, Paul Berry wrote:
On 22 February 2013 08:02, Michel Dänzer mic...@daenzer.net wrote:
From: Michel Dänzer michel.daen...@amd.com
Use LDS for calculating the deltas between neighbouring
pixels. Not sure the
sign
On Sam, 2013-02-23 at 05:37 +0100, Vinson Lee wrote:
Fixes out-of-bounds write reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
Reviewed-by: Michel Dänzer mic...@daenzer.net
--
Earthling Michel Dänzer | http://www.amd.com
Libre software
that's correct, but if it is, the change should be
amended with an explanation why.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
___
mesa-dev mailing
-key.export_16bpc cbuf)
0x1;
if (compressed)
Don't sneak in unrelated changes like this. You can push trivial fixes
like this without review.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X
On Die, 2013-03-05 at 15:27 +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
This could use a little more information, e.g.: What are LICM and DCE?
Why is it a good idea to enable them?
--
Earthling Michel
lib/gallium/swrast_dri.so
based on llvmpipe/softpipe.
Presumably both are built before and after the change(s) in question,
they merely changed which one ends up installed / picked up first at
runtime.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software
a bit more into the depth. For example imagine
a loop with an unused variable incremented, the increment depends on
itself, but it's completely useless...
Thanks for the explanations guys, but the point is really that the
change needs to be self-explanatory. :)
--
Earthling Michel Dänzer
101 - 200 of 2182 matches
Mail list logo