aboutsummaryrefslogtreecommitdiff
path: root/layers/core_validation.cpp
diff options
context:
space:
mode:
authorChris Forbes <chrisforbes@google.com>2017-04-24 11:22:45 -0700
committerChris Forbes <chrisf@ijw.co.nz>2017-04-25 07:39:07 +1200
commit1ac1eca2254c01322ad5ec4d3743d31d4cc32d9e (patch)
tree22e53eb39c051a803719f0fa54ee9029de56ded8 /layers/core_validation.cpp
parent2f820f736a4ca56a22f0147ed30151c14879d2d5 (diff)
downloadusermoji-1ac1eca2254c01322ad5ec4d3743d31d4cc32d9e.tar.xz
layers: Inline VkCommandBuffer overload of clear_cmd_buf_...
This was silly. On the very next line, we called GetCBNode again.
Diffstat (limited to 'layers/core_validation.cpp')
-rw-r--r--layers/core_validation.cpp6
1 files changed, 1 insertions, 5 deletions
diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp
index d7874c76..6e9a53a4 100644
--- a/layers/core_validation.cpp
+++ b/layers/core_validation.cpp
@@ -602,10 +602,6 @@ static void clear_cmd_buf_and_mem_references(layer_data *dev_data, GLOBAL_CB_NOD
cb_node->validate_functions.clear();
}
}
-// Overloaded call to above function when GLOBAL_CB_NODE has not already been looked-up
-static void clear_cmd_buf_and_mem_references(layer_data *dev_data, const VkCommandBuffer cb) {
- clear_cmd_buf_and_mem_references(dev_data, GetCBNode(dev_data, cb));
-}
// Clear a single object binding from given memory object, or report error if binding is missing
static bool ClearMemoryObjectBinding(layer_data *dev_data, uint64_t handle, VulkanObjectType type, VkDeviceMemory mem) {
@@ -5778,8 +5774,8 @@ static void PostCallRecordDestroyCommandPool(layer_data *dev_data, VkCommandPool
// Must remove cmdpool from cmdpoolmap, after removing all cmdbuffers in its list from the commandBufferMap
clearCommandBuffersInFlight(dev_data, cp_state);
for (auto cb : cp_state->commandBuffers) {
- clear_cmd_buf_and_mem_references(dev_data, cb);
auto cb_node = GetCBNode(dev_data, cb);
+ clear_cmd_buf_and_mem_references(dev_data, cb_node);
// Remove references to this cb_node prior to delete
// TODO : Need better solution here, resetCB?
for (auto obj : cb_node->object_bindings) {