Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-26 Thread Samuel Pitoiset
On 10/25/2017 10:58 PM, Bas Nieuwenhuizen wrote: On Wed, Oct 25, 2017 at 4:03 PM, Samuel Pitoiset wrote: On 10/25/2017 02:20 PM, Alex Smith wrote: On 25 October 2017 at 12:46, Samuel Pitoiset mailto:samuel.pitoi...@gmail.com>> wrote: I have something similar on my local tree (starte

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-26 Thread Alex Smith
On 25 October 2017 at 21:58, Bas Nieuwenhuizen wrote: > On Wed, Oct 25, 2017 at 4:03 PM, Samuel Pitoiset > wrote: > > > > > > On 10/25/2017 02:20 PM, Alex Smith wrote: > >> > >> On 25 October 2017 at 12:46, Samuel Pitoiset >> > wrote: > >> > >> I have somet

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-25 Thread Bas Nieuwenhuizen
On Wed, Oct 25, 2017 at 4:03 PM, Samuel Pitoiset wrote: > > > On 10/25/2017 02:20 PM, Alex Smith wrote: >> >> On 25 October 2017 at 12:46, Samuel Pitoiset > > wrote: >> >> I have something similar on my local tree (started on monday). >> >> Though, I don't

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-25 Thread Samuel Pitoiset
On 10/25/2017 02:20 PM, Alex Smith wrote: On 25 October 2017 at 12:46, Samuel Pitoiset > wrote: I have something similar on my local tree (started on monday). Though, I don't like the way we expose the number of VGPRS/SGPRS because we can't really

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-25 Thread Alex Smith
On 25 October 2017 at 12:46, Samuel Pitoiset wrote: > I have something similar on my local tree (started on monday). > > Though, I don't like the way we expose the number of VGPRS/SGPRS because > we can't really figure out the number of spilled ones. My assumption was that if we've spilled then

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-25 Thread Samuel Pitoiset
I have something similar on my local tree (started on monday). Though, I don't like the way we expose the number of VGPRS/SGPRS because we can't really figure out the number of spilled ones. On 10/25/2017 01:18 PM, Alex Smith wrote: This allows an app to query shader statistics and get a disa

[Mesa-dev] [PATCH 2/2] radv: Implement VK_AMD_shader_info

2017-10-25 Thread Alex Smith
This allows an app to query shader statistics and get a disassembly of a shader. RenderDoc git has support for it, so this allows you to view shader disassembly from a capture. When this extension is enabled on a device (or when tracing), we now disable pipeline caching, since we don't get the sha