diff options
author | Mark Lobodzinski <mark@lunarg.com> | 2019-03-06 16:11:56 -0700 |
---|---|---|
committer | Mark Lobodzinski <mark@lunarg.com> | 2019-03-07 11:22:45 -0700 |
commit | 2e49588c8d0c7ebb08d79aef702ddb2628abd805 (patch) | |
tree | 7d087f38c99f870bb5210eacc1f40ca2521023bb | |
parent | fd917d2dcba1f22d717e6dda588bb3bf35218cce (diff) | |
download | vulkan-validation-layers-2e49588c8d0c7ebb08d79aef702ddb2628abd805.tar.gz |
layers: Update RenderPassState accessor funcs
Change-Id: Ic3c9f198b0fcb4b7243495b11813f393d3d4b520
-rw-r--r-- | layers/buffer_validation.cpp | 6 | ||||
-rw-r--r-- | layers/core_validation.cpp | 36 | ||||
-rw-r--r-- | layers/core_validation.h | 4 |
3 files changed, 23 insertions, 23 deletions
diff --git a/layers/buffer_validation.cpp b/layers/buffer_validation.cpp index de36eada9..0b493f705 100644 --- a/layers/buffer_validation.cpp +++ b/layers/buffer_validation.cpp @@ -456,10 +456,10 @@ bool CoreChecks::VerifyFramebufferAndRenderPassLayouts(layer_data *device_data, GLOBAL_CB_NODE *pCB, const VkRenderPassBeginInfo *pRenderPassBegin, const FRAMEBUFFER_STATE *framebuffer_state) { bool skip = false; - auto const pRenderPassInfo = GetRenderPassState(device_data, pRenderPassBegin->renderPass)->createInfo.ptr(); + auto const pRenderPassInfo = GetRenderPassState(pRenderPassBegin->renderPass)->createInfo.ptr(); auto const &framebufferInfo = framebuffer_state->createInfo; - auto render_pass = GetRenderPassState(device_data, pRenderPassBegin->renderPass)->renderPass; + auto render_pass = GetRenderPassState(pRenderPassBegin->renderPass)->renderPass; auto framebuffer = framebuffer_state->framebuffer; if (pRenderPassInfo->attachmentCount != framebufferInfo.attachmentCount) { @@ -1176,7 +1176,7 @@ bool CoreChecks::VerifyImageLayout(layer_data const *device_data, GLOBAL_CB_NODE void CoreChecks::TransitionFinalSubpassLayouts(layer_data *device_data, GLOBAL_CB_NODE *pCB, const VkRenderPassBeginInfo *pRenderPassBegin, FRAMEBUFFER_STATE *framebuffer_state) { - auto renderPass = GetRenderPassState(device_data, pRenderPassBegin->renderPass); + auto renderPass = GetRenderPassState(pRenderPassBegin->renderPass); if (!renderPass) return; const VkRenderPassCreateInfo2KHR *pRenderPassInfo = renderPass->createInfo.ptr(); diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp index 7cd77dddd..b52e49496 100644 --- a/layers/core_validation.cpp +++ b/layers/core_validation.cpp @@ -644,17 +644,17 @@ PIPELINE_STATE *CoreChecks::GetPipelineState(VkPipeline pipeline) { return it->second.get(); } -RENDER_PASS_STATE *CoreChecks::GetRenderPassState(layer_data const *dev_data, VkRenderPass renderpass) { - auto it = dev_data->renderPassMap.find(renderpass); - if (it == dev_data->renderPassMap.end()) { +RENDER_PASS_STATE *CoreChecks::GetRenderPassState(VkRenderPass renderpass) { + auto it = renderPassMap.find(renderpass); + if (it == renderPassMap.end()) { return nullptr; } return it->second.get(); } -std::shared_ptr<RENDER_PASS_STATE> CoreChecks::GetRenderPassStateSharedPtr(layer_data const *dev_data, VkRenderPass renderpass) { - auto it = dev_data->renderPassMap.find(renderpass); - if (it == dev_data->renderPassMap.end()) { +std::shared_ptr<RENDER_PASS_STATE> CoreChecks::GetRenderPassStateSharedPtr(VkRenderPass renderpass) { + auto it = renderPassMap.find(renderpass); + if (it == renderPassMap.end()) { return nullptr; } return it->second; @@ -2059,7 +2059,7 @@ BASE_NODE *CoreChecks::GetStateStructPtrFromObject(layer_data *dev_data, VK_OBJE break; } case kVulkanObjectTypeRenderPass: { - base_ptr = GetRenderPassState(dev_data, reinterpret_cast<VkRenderPass &>(object_struct.handle)); + base_ptr = GetRenderPassState(reinterpret_cast<VkRenderPass &>(object_struct.handle)); break; } case kVulkanObjectTypeDeviceMemory: { @@ -4971,7 +4971,7 @@ void CoreChecks::PreCallRecordDestroyFramebuffer(VkDevice device, VkFramebuffer bool CoreChecks::PreCallValidateDestroyRenderPass(VkDevice device, VkRenderPass renderPass, const VkAllocationCallbacks *pAllocator) { layer_data *device_data = GetLayerDataPtr(get_dispatch_key(device), layer_data_map); - RENDER_PASS_STATE *rp_state = GetRenderPassState(device_data, renderPass); + RENDER_PASS_STATE *rp_state = GetRenderPassState(renderPass); VK_OBJECT obj_struct = {HandleToUint64(renderPass), kVulkanObjectTypeRenderPass}; bool skip = false; if (rp_state) { @@ -4984,7 +4984,7 @@ bool CoreChecks::PreCallValidateDestroyRenderPass(VkDevice device, VkRenderPass void CoreChecks::PreCallRecordDestroyRenderPass(VkDevice device, VkRenderPass renderPass, const VkAllocationCallbacks *pAllocator) { layer_data *device_data = GetLayerDataPtr(get_dispatch_key(device), layer_data_map); if (!renderPass) return; - RENDER_PASS_STATE *rp_state = GetRenderPassState(device_data, renderPass); + RENDER_PASS_STATE *rp_state = GetRenderPassState(renderPass); VK_OBJECT obj_struct = {HandleToUint64(renderPass), kVulkanObjectTypeRenderPass}; InvalidateCommandBuffers(device_data, rp_state->cb_bindings, obj_struct); device_data->renderPassMap.erase(renderPass); @@ -5192,7 +5192,7 @@ bool CoreChecks::PreCallValidateCreateGraphicsPipelines(VkDevice device, VkPipel for (uint32_t i = 0; i < count; i++) { cgpl_state->pipe_state.push_back(std::unique_ptr<PIPELINE_STATE>(new PIPELINE_STATE)); (cgpl_state->pipe_state)[i]->initGraphicsPipeline(&pCreateInfos[i], - GetRenderPassStateSharedPtr(device_data, pCreateInfos[i].renderPass)); + GetRenderPassStateSharedPtr(pCreateInfos[i].renderPass)); (cgpl_state->pipe_state)[i]->pipeline_layout = *GetPipelineLayout(device_data, pCreateInfos[i].layout); } @@ -6301,7 +6301,7 @@ bool CoreChecks::PreCallValidateBeginCommandBuffer(VkCommandBuffer commandBuffer // renderPass that framebuffer was created with must be compatible with local renderPass skip |= ValidateRenderPassCompatibility(device_data, "framebuffer", framebuffer->rp_state.get(), - "command buffer", GetRenderPassState(device_data, pInfo->renderPass), + "command buffer", GetRenderPassState(pInfo->renderPass), "vkBeginCommandBuffer()", "VUID-VkCommandBufferBeginInfo-flags-00055"); } } @@ -6317,7 +6317,7 @@ bool CoreChecks::PreCallValidateBeginCommandBuffer(VkCommandBuffer commandBuffer } } if (pInfo && pInfo->renderPass != VK_NULL_HANDLE) { - auto renderPass = GetRenderPassState(device_data, pInfo->renderPass); + auto renderPass = GetRenderPassState(pInfo->renderPass); if (renderPass) { if (pInfo->subpass >= renderPass->createInfo.subpassCount) { skip |= log_msg(device_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, @@ -6385,7 +6385,7 @@ void CoreChecks::PreCallRecordBeginCommandBuffer(VkCommandBuffer commandBuffer, // If we are a secondary command-buffer and inheriting. Update the items we should inherit. if ((cb_state->createInfo.level != VK_COMMAND_BUFFER_LEVEL_PRIMARY) && (cb_state->beginInfo.flags & VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT)) { - cb_state->activeRenderPass = GetRenderPassState(device_data, cb_state->beginInfo.pInheritanceInfo->renderPass); + cb_state->activeRenderPass = GetRenderPassState(cb_state->beginInfo.pInheritanceInfo->renderPass); cb_state->activeSubpass = cb_state->beginInfo.pInheritanceInfo->subpass; cb_state->activeFramebuffer = cb_state->beginInfo.pInheritanceInfo->framebuffer; cb_state->framebuffers.insert(cb_state->beginInfo.pInheritanceInfo->framebuffer); @@ -8979,7 +8979,7 @@ bool CoreChecks::MatchUsage(layer_data *dev_data, uint32_t count, const VkAttach bool CoreChecks::ValidateFramebufferCreateInfo(layer_data *dev_data, const VkFramebufferCreateInfo *pCreateInfo) { bool skip = false; - auto rp_state = GetRenderPassState(dev_data, pCreateInfo->renderPass); + auto rp_state = GetRenderPassState(pCreateInfo->renderPass); if (rp_state) { const VkRenderPassCreateInfo2KHR *rpci = rp_state->createInfo.ptr(); if (rpci->attachmentCount != pCreateInfo->attachmentCount) { @@ -9130,7 +9130,7 @@ void CoreChecks::PostCallRecordCreateFramebuffer(VkDevice device, const VkFrameb if (VK_SUCCESS != result) return; // Shadow create info and store in map std::unique_ptr<FRAMEBUFFER_STATE> fb_state( - new FRAMEBUFFER_STATE(*pFramebuffer, pCreateInfo, GetRenderPassStateSharedPtr(device_data, pCreateInfo->renderPass))); + new FRAMEBUFFER_STATE(*pFramebuffer, pCreateInfo, GetRenderPassStateSharedPtr(pCreateInfo->renderPass))); for (uint32_t i = 0; i < pCreateInfo->attachmentCount; ++i) { VkImageView view = pCreateInfo->pAttachments[i]; @@ -10233,7 +10233,7 @@ bool CoreChecks::ValidateCmdBeginRenderPass(layer_data *device_data, VkCommandBu RenderPassCreateVersion rp_version, const VkRenderPassBeginInfo *pRenderPassBegin) { GLOBAL_CB_NODE *cb_state = GetCBNode(commandBuffer); assert(cb_state); - auto render_pass_state = pRenderPassBegin ? GetRenderPassState(device_data, pRenderPassBegin->renderPass) : nullptr; + auto render_pass_state = pRenderPassBegin ? GetRenderPassState(pRenderPassBegin->renderPass) : nullptr; auto framebuffer = pRenderPassBegin ? GetFramebufferState(device_data, pRenderPassBegin->framebuffer) : nullptr; bool skip = false; @@ -10337,7 +10337,7 @@ bool CoreChecks::PreCallValidateCmdBeginRenderPass2KHR(VkCommandBuffer commandBu void CoreChecks::RecordCmdBeginRenderPassState(layer_data *device_data, VkCommandBuffer commandBuffer, const VkRenderPassBeginInfo *pRenderPassBegin, const VkSubpassContents contents) { GLOBAL_CB_NODE *cb_state = GetCBNode(commandBuffer); - auto render_pass_state = pRenderPassBegin ? GetRenderPassState(device_data, pRenderPassBegin->renderPass) : nullptr; + auto render_pass_state = pRenderPassBegin ? GetRenderPassState(pRenderPassBegin->renderPass) : nullptr; auto framebuffer = pRenderPassBegin ? GetFramebufferState(device_data, pRenderPassBegin->framebuffer) : nullptr; if (render_pass_state) { @@ -10593,7 +10593,7 @@ bool CoreChecks::PreCallValidateCmdExecuteCommands(VkCommandBuffer commandBuffer device_data->report_data->FormatHandle(pCommandBuffers[i]).c_str(), i); } else if (cb_state->activeRenderPass) { // Secondary CB w/i RenderPass must have *CONTINUE_BIT set if (sub_cb_state->beginInfo.pInheritanceInfo != nullptr) { - auto secondary_rp_state = GetRenderPassState(device_data, sub_cb_state->beginInfo.pInheritanceInfo->renderPass); + auto secondary_rp_state = GetRenderPassState(sub_cb_state->beginInfo.pInheritanceInfo->renderPass); if (!(sub_cb_state->beginInfo.flags & VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT)) { skip |= log_msg(device_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_COMMAND_BUFFER_EXT, HandleToUint64(pCommandBuffers[i]), diff --git a/layers/core_validation.h b/layers/core_validation.h index 03c0a4a53..7bf5f293e 100644 --- a/layers/core_validation.h +++ b/layers/core_validation.h @@ -257,8 +257,8 @@ class CoreChecks : public ValidationObject { SWAPCHAIN_NODE* GetSwapchainNode(VkSwapchainKHR); GLOBAL_CB_NODE* GetCBNode(const VkCommandBuffer cb); PIPELINE_STATE* GetPipelineState(VkPipeline pipeline); - RENDER_PASS_STATE* GetRenderPassState(layer_data const* dev_data, VkRenderPass renderpass); - std::shared_ptr<RENDER_PASS_STATE> GetRenderPassStateSharedPtr(layer_data const* dev_data, VkRenderPass renderpass); + RENDER_PASS_STATE* GetRenderPassState(VkRenderPass renderpass); + std::shared_ptr<RENDER_PASS_STATE> GetRenderPassStateSharedPtr(VkRenderPass renderpass); FRAMEBUFFER_STATE* GetFramebufferState(const layer_data* my_data, VkFramebuffer framebuffer); COMMAND_POOL_NODE* GetCommandPoolNode(layer_data* dev_data, VkCommandPool pool); shader_module const* GetShaderModuleState(layer_data const* dev_data, VkShaderModule module); |