descriptor updates with no writes or copies return early

vulkano-rs · Apr 20, 2024 · 2052dde · 2052dde
1 parent 3ad40ae
commit 2052dde
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/vulkano/src/descriptor_set/mod.rs b/vulkano/src/descriptor_set/mod.rs
@@ -191,6 +191,9 @@ impl DescriptorSet {
  ) -> Result<(), Box<ValidationError>> {
  let descriptor_writes: SmallVec<[_; 8]> = descriptor_writes.into_iter().collect();
  let descriptor_copies: SmallVec<[_; 8]> = descriptor_copies.into_iter().collect();
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return Ok(());
+ }
 
  self.inner
  .validate_update(&descriptor_writes, &descriptor_copies)?;
@@ -215,6 +218,9 @@ impl DescriptorSet {
  ) {
  let descriptor_writes: SmallVec<[_; 8]> = descriptor_writes.into_iter().collect();
  let descriptor_copies: SmallVec<[_; 8]> = descriptor_copies.into_iter().collect();
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return;
+ }
 
  Self::update_inner(
  &self.inner,
@@ -236,6 +242,9 @@ impl DescriptorSet {
  ) -> Result<(), Box<ValidationError>> {
  let descriptor_writes: SmallVec<[_; 8]> = descriptor_writes.into_iter().collect();
  let descriptor_copies: SmallVec<[_; 8]> = descriptor_copies.into_iter().collect();
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return Ok(());
+ }
 
  self.inner
  .validate_update(&descriptor_writes, &descriptor_copies)?;
@@ -258,6 +267,9 @@ impl DescriptorSet {
  ) {
  let descriptor_writes: SmallVec<[_; 8]> = descriptor_writes.into_iter().collect();
  let descriptor_copies: SmallVec<[_; 8]> = descriptor_copies.into_iter().collect();
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return;
+ }
 
  Self::update_inner(
  &self.inner,

diff --git a/vulkano/src/descriptor_set/sys.rs b/vulkano/src/descriptor_set/sys.rs
@@ -86,6 +86,10 @@ impl RawDescriptorSet {
  descriptor_writes: &[WriteDescriptorSet],
  descriptor_copies: &[CopyDescriptorSet],
  ) -> Result<(), Box<ValidationError>> {
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return Ok(());
+ }
+
  self.validate_update(descriptor_writes, descriptor_copies)?;
 
  self.update_unchecked(descriptor_writes, descriptor_copies);
@@ -117,6 +121,10 @@ impl RawDescriptorSet {
  descriptor_writes: &[WriteDescriptorSet],
  descriptor_copies: &[CopyDescriptorSet],
  ) {
+ if descriptor_writes.is_empty() && descriptor_copies.is_empty() {
+ return;
+ }
+
  struct PerDescriptorWrite {
  write_info: DescriptorWriteInfo,
  acceleration_structures: ash::vk::WriteDescriptorSetAccelerationStructureKHR<'static>,