<div dir="ltr">Reviewed-by: Alex Smith <<a href="mailto:asmith@feralinteractive.com">asmith@feralinteractive.com</a>></div><div class="gmail_extra"><br><div class="gmail_quote">On 15 June 2018 at 17:52, Eric Engestrom <span dir="ltr"><<a href="mailto:eric.engestrom@intel.com" target="_blank">eric.engestrom@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It's a bit late to round up after an integer division.<br>
<br>
Fixes: de889794134e6245e08a2 "radv: Implement VK_AMD_shader_info"<br>
Cc: Alex Smith <<a href="mailto:asmith@feralinteractive.com">asmith@feralinteractive.com</a>><br>
Signed-off-by: Eric Engestrom <<a href="mailto:eric.engestrom@intel.com">eric.engestrom@intel.com</a>><br>
---<br>
src/amd/vulkan/radv_shader.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c<br>
index 76790a19047a86abdad5..<wbr>b31eb9bfda5e9e29e115 100644<br>
--- a/src/amd/vulkan/radv_shader.c<br>
+++ b/src/amd/vulkan/radv_shader.c<br>
@@ -808,7 +808,7 @@ radv_GetShaderInfoAMD(VkDevice _device,<br>
unsigned workgroup_size = local_size[0] * local_size[1] * local_size[2];<br>
<br>
statistics.numAvailableVgprs = statistics.numPhysicalVgprs /<br>
- ceil(workgroup_size / statistics.numPhysicalVgprs);<br>
+ ceil((double)workgroup_size / statistics.numPhysicalVgprs);<br>
<br>
statistics.<wbr>computeWorkGroupSize[0] = local_size[0];<br>
statistics.<wbr>computeWorkGroupSize[1] = local_size[1];<br>
<span class="HOEnZb"><font color="#888888">-- <br>
Cheers,<br>
Eric<br>
<br>
</font></span></blockquote></div><br></div>