[Mesa-dev] [PATCH] R600/SI: Add patterns for V_MAD(_LEGACY) instructions.

Michel Dänzer michel at daenzer.net
Tue Jan 22 07:06:05 PST 2013


On Mon, 2013-01-21 at 23:03 +0100, Tom Stellard wrote: 
> 
> I don't think we emit the int_AMDGPU_mul intrinsic anymore, but it probably
> doesn't hurt to keep it around until we sort out all of the legacy vs non-legacy
> instruction issues.

Right. BTW, emitting int_AMDGPU_mad for TGSI_OPCODE_MAD doesn't fix the
glxgears problem I reported on IRC with current master (in contrast to
0ad1fefd6951aa47ab58a41dc9ee73083cbcf85c, where that problem was
introduced). Looks like the intrinsic ends up generating cmp/select
pairs instead of V_MAD in some cases now, which breaks somehow.


In the meantime, I've come to realize that this patch breaks a bunch of
piglit tests in the glsl-1.[12]0/execution/variable-indexing groups (I
previously thought those tests were broken by the array uniform related
changes). I'm attaching a diff of the generated code. Not sure if this
is some kind of bad interaction between this change and the control flow
code, or another instance of tests which passed accidentally.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: MAD.diff
Type: text/x-patch
Size: 92687 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130122/47370638/attachment-0001.bin>


More information about the mesa-dev mailing list