Module: Mesa
Branch: master
Commit: cd60c995a6d26b5f1952f21482177f36e7ad5630
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=cd60c995a6d26b5f1952f21482177f36e7ad5630

Author: Jason Ekstrand <[email protected]>
Date:   Thu Feb 14 14:36:49 2019 -0600

anv/blorp: Delete a pointless assert

Just a little higher up in the function we assert that the aspect masks
are actually equal so there's no reason for the weaker check.  Also, the
temporary variables were causing compiler warnings in release builds.

Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>

---

 src/intel/vulkan/anv_blorp.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c
index f4eaf8f4db..0d3d3f948e 100644
--- a/src/intel/vulkan/anv_blorp.c
+++ b/src/intel/vulkan/anv_blorp.c
@@ -1276,11 +1276,6 @@ void anv_CmdResolveImage(
       const uint32_t layer_count =
          anv_get_layerCount(dst_image, &pRegions[r].dstSubresource);
 
-      VkImageAspectFlags src_mask = pRegions[r].srcSubresource.aspectMask;
-      VkImageAspectFlags dst_mask = pRegions[r].dstSubresource.aspectMask;
-
-      assert(anv_image_aspects_compatible(src_mask, dst_mask));
-
       uint32_t aspect_bit;
       anv_foreach_image_aspect_bit(aspect_bit, src_image,
                                    pRegions[r].srcSubresource.aspectMask) {

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to