aboutsummaryrefslogtreecommitdiff
path: root/layers/core_validation.cpp
diff options
context:
space:
mode:
authorChris Forbes <chrisforbes@google.com>2016-05-17 11:02:24 +1200
committerTobin Ehlis <tobine@google.com>2016-05-19 08:48:42 -0600
commit818c4b85891f8bded09b4d1178e508d39cb89bac (patch)
tree85192c6a3e9da4a59a0d68a4cba21012664bc1ad /layers/core_validation.cpp
parentd72b9acd6d857ee6e3e46f92f5f48c24b278923a (diff)
downloadusermoji-818c4b85891f8bded09b4d1178e508d39cb89bac.tar.xz
layers: Unify cb lookup in CmdBeginRenderPass
We already have a GLOBAL_CB_NODE ptr here, and we know it's not null. Signed-off-by: Chris Forbes <chrisforbes@google.com>
Diffstat (limited to 'layers/core_validation.cpp')
-rw-r--r--layers/core_validation.cpp45
1 files changed, 18 insertions, 27 deletions
diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp
index 934a2288..0715fda1 100644
--- a/layers/core_validation.cpp
+++ b/layers/core_validation.cpp
@@ -8857,40 +8857,31 @@ CmdBeginRenderPass(VkCommandBuffer commandBuffer, const VkRenderPassBeginInfo *p
if (pass_data != dev_data->renderPassMap.end()) {
RENDER_PASS_NODE* pRPNode = pass_data->second;
pCB->activeFramebuffer = pRenderPassBegin->framebuffer;
- auto cb_data = dev_data->commandBufferMap.find(commandBuffer);
for (size_t i = 0; i < pRPNode->attachments.size(); ++i) {
MT_FB_ATTACHMENT_INFO &fb_info = dev_data->frameBufferMap[pRenderPassBegin->framebuffer].attachments[i];
if (pRPNode->attachments[i].load_op == VK_ATTACHMENT_LOAD_OP_CLEAR) {
- if (cb_data != dev_data->commandBufferMap.end()) {
- std::function<bool()> function = [=]() {
- set_memory_valid(dev_data, fb_info.mem, true, fb_info.image);
- return false;
- };
- cb_data->second->validate_functions.push_back(function);
- }
+ std::function<bool()> function = [=]() {
+ set_memory_valid(dev_data, fb_info.mem, true, fb_info.image);
+ return false;
+ };
+ pCB->validate_functions.push_back(function);
} else if (pRPNode->attachments[i].load_op == VK_ATTACHMENT_LOAD_OP_DONT_CARE) {
- if (cb_data != dev_data->commandBufferMap.end()) {
- std::function<bool()> function = [=]() {
- set_memory_valid(dev_data, fb_info.mem, false, fb_info.image);
- return false;
- };
- cb_data->second->validate_functions.push_back(function);
- }
+ std::function<bool()> function = [=]() {
+ set_memory_valid(dev_data, fb_info.mem, false, fb_info.image);
+ return false;
+ };
+ pCB->validate_functions.push_back(function);
} else if (pRPNode->attachments[i].load_op == VK_ATTACHMENT_LOAD_OP_LOAD) {
- if (cb_data != dev_data->commandBufferMap.end()) {
- std::function<bool()> function = [=]() {
- return validate_memory_is_valid(dev_data, fb_info.mem, "vkCmdBeginRenderPass()", fb_info.image);
- };
- cb_data->second->validate_functions.push_back(function);
- }
+ std::function<bool()> function = [=]() {
+ return validate_memory_is_valid(dev_data, fb_info.mem, "vkCmdBeginRenderPass()", fb_info.image);
+ };
+ pCB->validate_functions.push_back(function);
}
if (pRPNode->attachment_first_read[pRPNode->attachments[i].attachment]) {
- if (cb_data != dev_data->commandBufferMap.end()) {
- std::function<bool()> function = [=]() {
- return validate_memory_is_valid(dev_data, fb_info.mem, "vkCmdBeginRenderPass()", fb_info.image);
- };
- cb_data->second->validate_functions.push_back(function);
- }
+ std::function<bool()> function = [=]() {
+ return validate_memory_is_valid(dev_data, fb_info.mem, "vkCmdBeginRenderPass()", fb_info.image);
+ };
+ pCB->validate_functions.push_back(function);
}
}
}