Mesa (master): intel/blorp: Use documented RECTLIST vertex positions

Nanley Chery nchery at kemper.freedesktop.org
Wed Oct 5 16:41:54 UTC 2016


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

Author: Nanley Chery <nanley.g.chery at intel.com>
Date:   Wed Oct  5 09:32:52 2016 -0700

intel/blorp: Use documented RECTLIST vertex positions

Use the vertex positions described in the PRMs. This has no effect on
rendering but quiets the simulator warnings seen when the vertices
appear out of order.

Signed-off-by: Nanley Chery <nanley.g.chery at intel.com>
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
Reviewed-by: Ben Widawsky <ben at bwidawsk.net>

---

 src/intel/blorp/blorp_genX_exec.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/intel/blorp/blorp_genX_exec.h b/src/intel/blorp/blorp_genX_exec.h
index eb4a5b9..62f16a3 100644
--- a/src/intel/blorp/blorp_genX_exec.h
+++ b/src/intel/blorp/blorp_genX_exec.h
@@ -171,8 +171,8 @@ blorp_emit_vertex_data(struct blorp_batch *batch,
                        uint32_t *size)
 {
    const float vertices[] = {
-      /* v0 */ (float)params->x0, (float)params->y1,
-      /* v1 */ (float)params->x1, (float)params->y1,
+      /* v0 */ (float)params->x1, (float)params->y1,
+      /* v1 */ (float)params->x0, (float)params->y1,
       /* v2 */ (float)params->x0, (float)params->y0,
    };
 
@@ -287,7 +287,7 @@ blorp_emit_vertex_elements(struct blorp_batch *batch,
     *   v2 ------ implied
     *    |        |
     *    |        |
-    *   v0 ----- v1
+    *   v1 ----- v0
     *
     * Since the VS is disabled, the clipper loads each VUE directly from
     * the URB. This is controlled by the 3DSTATE_VERTEX_BUFFERS and




More information about the mesa-commit mailing list