anv/cmd_buffer: Return a VkResult from verify_cmd_parser
This fixes a "statement with no effect" compiler warning Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
This commit is contained in:
@@ -1899,18 +1899,17 @@ genX(cmd_buffer_flush_compute_state)(struct anv_cmd_buffer *cmd_buffer)
|
|||||||
|
|
||||||
#if GEN_GEN == 7
|
#if GEN_GEN == 7
|
||||||
|
|
||||||
static bool
|
static VkResult
|
||||||
verify_cmd_parser(const struct anv_device *device,
|
verify_cmd_parser(const struct anv_device *device,
|
||||||
int required_version,
|
int required_version,
|
||||||
const char *function)
|
const char *function)
|
||||||
{
|
{
|
||||||
if (device->instance->physicalDevice.cmd_parser_version < required_version) {
|
if (device->instance->physicalDevice.cmd_parser_version < required_version) {
|
||||||
vk_errorf(VK_ERROR_FEATURE_NOT_PRESENT,
|
return vk_errorf(VK_ERROR_FEATURE_NOT_PRESENT,
|
||||||
"cmd parser version %d is required for %s",
|
"cmd parser version %d is required for %s",
|
||||||
required_version, function);
|
required_version, function);
|
||||||
return false;
|
|
||||||
} else {
|
} else {
|
||||||
return true;
|
return VK_SUCCESS;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1981,7 +1980,8 @@ void genX(CmdDispatchIndirect)(
|
|||||||
/* Linux 4.4 added command parser version 5 which allows the GPGPU
|
/* Linux 4.4 added command parser version 5 which allows the GPGPU
|
||||||
* indirect dispatch registers to be written.
|
* indirect dispatch registers to be written.
|
||||||
*/
|
*/
|
||||||
if (!verify_cmd_parser(cmd_buffer->device, 5, "vkCmdDispatchIndirect"))
|
if (verify_cmd_parser(cmd_buffer->device, 5,
|
||||||
|
"vkCmdDispatchIndirect") != VK_SUCCESS)
|
||||||
return;
|
return;
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user