[Mesa-dev] [PATCH 07/10] radeonsi: don't use the AMDGPU intrinsic for CMP

Marek Olšák maraeo at gmail.com
Sun Oct 11 15:07:39 PDT 2015


On Sun, Oct 11, 2015 at 6:48 PM, Matt Arsenault <arsenm2 at gmail.com> wrote:
>
> On Oct 10, 2015, at 6:29 PM, Marek Olšák <maraeo at gmail.com> wrote:
>
> The increase in VGPRs in unfortunate, but the decrease in the scratch size
> is always welcome.

Oh sorry, the stats are actually for the previous patch replacing the
AMDGPU.lrp with standard opcodes.

This patch has no effect on the compiler.

> Do you have a specific example where this happens you can post?

Attached are good and bad versions, and a diff (view with "less -R").

Marek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vgpr_bad
Type: application/octet-stream
Size: 16665 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20151012/745817a2/attachment-0003.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vgpr_good
Type: application/octet-stream
Size: 16397 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20151012/745817a2/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vgpr_asm_diff
Type: application/octet-stream
Size: 18162 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20151012/745817a2/attachment-0005.obj>


More information about the mesa-dev mailing list