Re: [Mesa-dev] [PATCH 7/7] radv: add support for 32-bit pointers in user data SGPRs

2018-05-20 Thread Bas Nieuwenhuizen
On Sun, May 20, 2018 at 3:27 PM, Samuel Pitoiset wrote: > > > On 05/19/2018 06:22 PM, Bas Nieuwenhuizen wrote: >> >> On Fri, May 18, 2018 at 6:00 PM, Samuel Pitoiset >> wrote: >>> >>> We still use 64-bit GPU pointers for all ring buffers because >>> llvm.amdgcn.implicit.buffer.ptr doesn't seem to

Re: [Mesa-dev] [PATCH 7/7] radv: add support for 32-bit pointers in user data SGPRs

2018-05-20 Thread Samuel Pitoiset
On 05/19/2018 06:22 PM, Bas Nieuwenhuizen wrote: On Fri, May 18, 2018 at 6:00 PM, Samuel Pitoiset wrote: We still use 64-bit GPU pointers for all ring buffers because llvm.amdgcn.implicit.buffer.ptr doesn't seem to support 32-bit GPU pointers for now. This can be improved later anyways. Vega

Re: [Mesa-dev] [PATCH 7/7] radv: add support for 32-bit pointers in user data SGPRs

2018-05-19 Thread Bas Nieuwenhuizen
On Fri, May 18, 2018 at 6:00 PM, Samuel Pitoiset wrote: > We still use 64-bit GPU pointers for all ring buffers because > llvm.amdgcn.implicit.buffer.ptr doesn't seem to support 32-bit > GPU pointers for now. This can be improved later anyways. > > Vega10: > Totals from affected shaders: > SGPRS:

[Mesa-dev] [PATCH 7/7] radv: add support for 32-bit pointers in user data SGPRs

2018-05-18 Thread Samuel Pitoiset
We still use 64-bit GPU pointers for all ring buffers because llvm.amdgcn.implicit.buffer.ptr doesn't seem to support 32-bit GPU pointers for now. This can be improved later anyways. Vega10: Totals from affected shaders: SGPRS: 1008722 -> 1026710 (1.78 %) VGPRS: 706580 -> 707136 (0.08 %) Spilled S