diff options
| author | Jason Ekstrand <jason.ekstrand@intel.com> | 2017-02-10 20:47:18 -0800 |
|---|---|---|
| committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2017-02-14 07:50:13 -0800 |
| commit | 0ef14cdc980549cc7e7bafd06aaad356da7137ff (patch) | |
| tree | a8ad4bd6a4d202e34c3eb647e2aeb5e8f260cb5c | |
| parent | 956556b3c30ce3d38d0af795f9383df3bc2cf8a2 (diff) | |
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>
| -rw-r--r-- | src/intel/vulkan/genX_cmd_buffer.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 8db26e99121..d07fe78b52f 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/intel/vulkan/genX_cmd_buffer.c @@ -1899,18 +1899,17 @@ genX(cmd_buffer_flush_compute_state)(struct anv_cmd_buffer *cmd_buffer) #if GEN_GEN == 7 -static bool +static VkResult verify_cmd_parser(const struct anv_device *device, int required_version, const char *function) { if (device->instance->physicalDevice.cmd_parser_version < required_version) { - vk_errorf(VK_ERROR_FEATURE_NOT_PRESENT, - "cmd parser version %d is required for %s", - required_version, function); - return false; + return vk_errorf(VK_ERROR_FEATURE_NOT_PRESENT, + "cmd parser version %d is required for %s", + required_version, function); } 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 * 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; #endif |