Mesa (main): llvmpipe: consolidate legacy_points and multisample state

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Jul 25 10:56:32 UTC 2022


Module: Mesa
Branch: main
Commit: 8f99bab70ccbe70977d5a65c28fd427e6b1cd1e4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8f99bab70ccbe70977d5a65c28fd427e6b1cd1e4

Author: Erik Faye-Lund <erik.faye-lund at collabora.com>
Date:   Fri Jul  9 17:02:45 2021 +0200

llvmpipe: consolidate legacy_points and multisample state

This just makes the code a bit easier to follow.

Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Roland Scheidegger <sroland at vmware.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17653>

---

 src/gallium/drivers/llvmpipe/lp_setup.c       | 2 +-
 src/gallium/drivers/llvmpipe/lp_setup_point.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/gallium/drivers/llvmpipe/lp_setup.c b/src/gallium/drivers/llvmpipe/lp_setup.c
index 28dea4f82b3..a364cff6aba 100644
--- a/src/gallium/drivers/llvmpipe/lp_setup.c
+++ b/src/gallium/drivers/llvmpipe/lp_setup.c
@@ -622,7 +622,7 @@ lp_setup_bind_rasterizer( struct lp_setup_context *setup,
    setup->sprite_coord_origin = rast->sprite_coord_mode;
    setup->point_tri_clip = rast->point_size_per_vertex;
    setup->point_size_per_vertex = rast->point_size_per_vertex;
-   setup->legacy_points = !rast->point_quad_rasterization;
+   setup->legacy_points = !rast->point_quad_rasterization && !setup->multisample;
 }
 
 
diff --git a/src/gallium/drivers/llvmpipe/lp_setup_point.c b/src/gallium/drivers/llvmpipe/lp_setup_point.c
index d07d3a40c46..c34c82c0ec6 100644
--- a/src/gallium/drivers/llvmpipe/lp_setup_point.c
+++ b/src/gallium/drivers/llvmpipe/lp_setup_point.c
@@ -372,7 +372,7 @@ try_setup_point( struct lp_setup_context *setup,
       print_point(setup, v0, size);
 
    /* Bounding rectangle (in pixels) */
-   if (!setup->legacy_points || setup->multisample) {
+   if (!setup->legacy_points) {
       /*
        * Rasterize points as quads.
        */
@@ -535,7 +535,7 @@ try_setup_point( struct lp_setup_context *setup,
       plane[3].c = MIN2(y[1], (bbox.y1 + 1) << 8);
       plane[3].eo = 0;
 
-      if (!setup->legacy_points || setup->multisample) {
+      if (!setup->legacy_points) {
          /* adjust for fill-rule*/
          plane[0].c++; /* left */
          if (setup->bottom_edge_rule == 0)



More information about the mesa-commit mailing list