Reviewed-by: Alex Smith <[email protected]> On 15 June 2018 at 17:52, Eric Engestrom <[email protected]> wrote:
> It's a bit late to round up after an integer division. > > Fixes: de889794134e6245e08a2 "radv: Implement VK_AMD_shader_info" > Cc: Alex Smith <[email protected]> > Signed-off-by: Eric Engestrom <[email protected]> > --- > src/amd/vulkan/radv_shader.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c > index 76790a19047a86abdad5..b31eb9bfda5e9e29e115 100644 > --- a/src/amd/vulkan/radv_shader.c > +++ b/src/amd/vulkan/radv_shader.c > @@ -808,7 +808,7 @@ radv_GetShaderInfoAMD(VkDevice _device, > unsigned workgroup_size = local_size[0] * > local_size[1] * local_size[2]; > > statistics.numAvailableVgprs = > statistics.numPhysicalVgprs / > - > ceil(workgroup_size / statistics.numPhysicalVgprs); > + > ceil((double)workgroup_size / statistics.numPhysicalVgprs); > > statistics.computeWorkGroupSize[0] = > local_size[0]; > statistics.computeWorkGroupSize[1] = > local_size[1]; > -- > Cheers, > Eric > >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
