aboutsummaryrefslogtreecommitdiff
path: root/layers/descriptor_sets.cpp
diff options
context:
space:
mode:
authorTobin Ehlis <tobine@google.com>2017-01-03 14:09:57 -0700
committerTobin Ehlis <tobine@google.com>2017-01-04 06:49:18 -0700
commit236755c9702bcbdddd887a8ae3c0dafd80d569e6 (patch)
tree82598158c456efe5f12914238b331b1e9a4dfd3f /layers/descriptor_sets.cpp
parentbda40694f006c2aa082a1040ff7e7a163080f40d (diff)
downloadusermoji-236755c9702bcbdddd887a8ae3c0dafd80d569e6.tar.xz
scripts:Exceptions for duplicate unique ids
Added a number of exceptions for duplicate uses of unique error enums that are valid. Also added some comments to a few cases that need new unique ids.
Diffstat (limited to 'layers/descriptor_sets.cpp')
-rw-r--r--layers/descriptor_sets.cpp6
1 files changed, 4 insertions, 2 deletions
diff --git a/layers/descriptor_sets.cpp b/layers/descriptor_sets.cpp
index ccfad829..902a71cc 100644
--- a/layers/descriptor_sets.cpp
+++ b/layers/descriptor_sets.cpp
@@ -596,7 +596,7 @@ bool cvdescriptorset::DescriptorSet::ValidateCopyUpdate(const debug_report_data
std::string *error_msg) {
// Verify idle ds
if (in_use.load()) {
- // TODO : Re-using Allocate Idle error code, need copy update idle error code
+ // TODO : Re-using Free Idle error code, need copy update idle error code
*error_code = VALIDATION_ERROR_00919;
std::stringstream error_str;
error_str << "Cannot call vkUpdateDescriptorSets() to perform copy update on descriptor set " << set_
@@ -763,6 +763,8 @@ bool cvdescriptorset::ValidateImageUpdate(VkImageView image_view, VkImageLayout
format = image_node->createInfo.format;
usage = image_node->createInfo.usage;
// Validate that memory is bound to image
+ // TODO: This should have its own valid usage id apart from 2524 which is from CreateImageView case. The only
+ // the error here occurs is if memory bound to a created imageView has been freed.
if (ValidateMemoryIsBoundToImage(dev_data, image_node, "vkUpdateDescriptorSets()", VALIDATION_ERROR_02524)) {
*error_code = VALIDATION_ERROR_02524;
*error_msg = "No memory bound to image.";
@@ -1169,7 +1171,7 @@ bool cvdescriptorset::DescriptorSet::ValidateWriteUpdate(const debug_report_data
UNIQUE_VALIDATION_ERROR_CODE *error_code, std::string *error_msg) {
// Verify idle ds
if (in_use.load()) {
- // TODO : Re-using Allocate Idle error code, need write update idle error code
+ // TODO : Re-using Free Idle error code, need write update idle error code
*error_code = VALIDATION_ERROR_00919;
std::stringstream error_str;
error_str << "Cannot call vkUpdateDescriptorSets() to perform write update on descriptor set " << set_