commit 754338b569c4b899e64826f9b93cbeaccb15ba55
author Corentin Wallez <cwallez@chromium.org> Tue Oct 03 19:36:23 2023 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Oct 03 19:36:23 2023 +0000
tree a01fd1ac5f74d9334abc24a85706f4e011b966b4
parent 3d8b48e023f44645d8463868be8ba0ff539ebe81

Remove incorrect ASSERT that 1D T2T copy origin have z = 0

This is not correct because an empty 1D copy could have z = 1 and
copySize.depthOrArrayLayers = 0, which is valid.

Bug: chromium:1488363
Change-Id: Idc7737783b4e89428fd01e7e9a051df082a44fe0
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/154342
Reviewed-by: Loko Kung <lokokung@google.com>
Kokoro: Kokoro <noreply+kokoro@google.com>
Auto-Submit: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>

src/dawn/native/CommandValidation.cpp

diff --git a/src/dawn/native/CommandValidation.cpp b/src/dawn/native/CommandValidation.cpp
index 8ad5179..609d3c8 100644
--- a/src/dawn/native/CommandValidation.cpp
+++ b/src/dawn/native/CommandValidation.cpp
@@ -487,7 +487,7 @@
     if (src.texture == dst.texture) {
         switch (src.texture->GetDimension()) {
             case wgpu::TextureDimension::e1D:
-                DAWN_ASSERT(src.mipLevel == 0 && src.origin.z == 0 && dst.origin.z == 0);
+                DAWN_ASSERT(src.mipLevel == 0);
                 return DAWN_VALIDATION_ERROR("Copy is from %s to itself.", src.texture);
 
             case wgpu::TextureDimension::e2D: