Closed Silverlan closed 5 years ago
Something else related I just noticed: Both _PipelineCache::getdata and PipelineCache::merge use _anvilassert to check the VkResult, I believe that should either be _anvil_assert_vk_callsucceeded, or the assert should be removed altogether, since in both cases the result is checked again just below the assert:
anvil_assert(result_vk);
return is_vk_call_successful(result_vk);
Agreed, this is a shameful piece of nonsensical code :-) I've fixed this internally; will be including the change in the update, hopefully coming tomorrow.
_PipelineCache::getdata takes a void** for the output data, which is then implicitly cast to a void* in the vkGetPipelineCacheData-call, that doesn't seem right to me:
I'm not sure if this is a bug, or if there's some logic behind it I don't understand.