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

Author: Bas Nieuwenhuizen <[email protected]>
Date:   Fri Aug 25 00:26:44 2017 +0200

radv: Remove some intel comments from the resolve code.

These are clearly not applicable to radv.

Reviewed-by: Kenneth Graunke <[email protected]>

---

 src/amd/vulkan/radv_meta_resolve.c    | 7 -------
 src/amd/vulkan/radv_meta_resolve_cs.c | 7 -------
 src/amd/vulkan/radv_meta_resolve_fs.c | 7 -------
 3 files changed, 21 deletions(-)

diff --git a/src/amd/vulkan/radv_meta_resolve.c 
b/src/amd/vulkan/radv_meta_resolve.c
index 6023e0f899..dd811c2514 100644
--- a/src/amd/vulkan/radv_meta_resolve.c
+++ b/src/amd/vulkan/radv_meta_resolve.c
@@ -612,13 +612,6 @@ radv_cmd_buffer_resolve_subpass(struct radv_cmd_buffer 
*cmd_buffer)
 
                radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, 
false);
 
-               /* Subpass resolves must respect the render area. We can ignore 
the
-                * render area here because vkCmdBeginRenderPass set the render 
area
-                * with 3DSTATE_DRAWING_RECTANGLE.
-                *
-                * XXX(chadv): Does the hardware really respect
-                * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
-                */
                emit_resolve(cmd_buffer,
                             &(VkOffset2D) { 0, 0 },
                             &(VkExtent2D) { fb->width, fb->height });
diff --git a/src/amd/vulkan/radv_meta_resolve_cs.c 
b/src/amd/vulkan/radv_meta_resolve_cs.c
index d20d04231e..da6ca76b6d 100644
--- a/src/amd/vulkan/radv_meta_resolve_cs.c
+++ b/src/amd/vulkan/radv_meta_resolve_cs.c
@@ -529,13 +529,6 @@ radv_cmd_buffer_resolve_subpass_cs(struct radv_cmd_buffer 
*cmd_buffer)
 
                radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, 
false);
 
-               /* Subpass resolves must respect the render area. We can ignore 
the
-                * render area here because vkCmdBeginRenderPass set the render 
area
-                * with 3DSTATE_DRAWING_RECTANGLE.
-                *
-                * XXX(chadv): Does the hardware really respect
-                * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
-                */
                emit_resolve(cmd_buffer,
                             src_iview,
                             dst_iview,
diff --git a/src/amd/vulkan/radv_meta_resolve_fs.c 
b/src/amd/vulkan/radv_meta_resolve_fs.c
index 2f745f0ea0..373dd9665a 100644
--- a/src/amd/vulkan/radv_meta_resolve_fs.c
+++ b/src/amd/vulkan/radv_meta_resolve_fs.c
@@ -633,13 +633,6 @@ radv_cmd_buffer_resolve_subpass_fs(struct radv_cmd_buffer 
*cmd_buffer)
 
                radv_cmd_buffer_set_subpass(cmd_buffer, &resolve_subpass, 
false);
 
-               /* Subpass resolves must respect the render area. We can ignore 
the
-                * render area here because vkCmdBeginRenderPass set the render 
area
-                * with 3DSTATE_DRAWING_RECTANGLE.
-                *
-                * XXX(chadv): Does the hardware really respect
-                * 3DSTATE_DRAWING_RECTANGLE when draing a 3DPRIM_RECTLIST?
-                */
                emit_resolve(cmd_buffer,
                             src_iview,
                             dest_iview,

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

Reply via email to