URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=aae0c88797e7e44c55873b3e97cceed5c6e6cded Author: Ville Syrjälä <ville.syrj...@linux.intel.com> Date: Mon Mar 23 14:47:28 2015 +0200
i915: Make hw_prim[] const The table used to map the GL primitive to the hw primitive never changes so make it const. Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=84fec757de2f8c6ba47f2f56077993d62538711a Author: Ville Syrjälä <ville.syrj...@linux.intel.com> Date: Mon Mar 23 14:47:23 2015 +0200 t_dd_dmatmp: Make the render_tab[]s const These tables hold function pointers and they never change so make them const. Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=abbaf3301f2405adf61d2804706c00a391a1336e Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:57:15 2015 -0700 mesa: Remove unused HAVE_TRI_STRIP_1 defines Defined to 0 in a few places, but it's not used anywhere. Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d830965057bc01734dcca415969ebb6950f2e58c Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:29:50 2015 -0700 t_dd_dmatmp: Constify dmasz Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8e9968f184dd1f9727d1095aa45936d5c6ec9f16 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:26:10 2015 -0700 t_dd_dmatmp: Silence comparison between signed and unsigned integer expression warnings ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:83:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:83:55: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:116:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:116:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:140:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:140:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h: In function 'intel_render_line_loop_verts': ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:174:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:174:55: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:224:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:224:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:255:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:255:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:281:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:281:56: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h: In function 'intel_render_poly_verts': ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:313:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:313:59: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:365:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - nr); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:365:56: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - nr); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:83:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:83:55: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:116:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:116:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:140:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:140:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h: In function 'radeon_dma_render_line_loop_verts': ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:174:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:174:55: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:224:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:224:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:255:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:255:52: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:281:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:281:56: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h: In function 'radeon_dma_render_poly_verts': ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:313:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:313:59: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - j + 1); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:365:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] nr = MIN2(currentsz, count - nr); ^ ../../../../../src/mesa/tnl_dd/t_dd_dmatmp.h:365:56: warning: signed and unsigned type in conditional expression [-Wsign-compare] nr = MIN2(currentsz, count - nr); ^ Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d663d8f5d4088407af982577be075fe7e79d4e3d Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:12:06 2015 -0700 t_dd_dmatmp: Use stdbool.h No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b7259fc6b0c3a94d2ddbea74470e551c877fc324 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:10:05 2015 -0700 t_dd_dmatmp: General indentation and formatting fixes No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=57ae5c237d65b7283964f61779fcb07096da8b5b Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 17:04:33 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_ELTS change No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=25b42f13bd762119eb1dae565e999c1bd52f7c2d Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 16:57:32 2015 -0700 t_dd_dmatmp: Remove HAVE_ELTS support Two drivers use this file, and neither supports ELTs. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1f374958fdc02050e9d3c0f2dfd0b87702470fb7 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:35:51 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_TRI_FANS change No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=03c3208c185c29dd26446ffa5bae8f05f51d3f1b Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:31:09 2015 -0700 t_dd_dmatmp: Require HAVE_TRI_FANS Two drivers use this file, and both support triangle fans. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2e19ed3cb54cb33e8d772bfde78cc1d33dc0853b Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:29:31 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_TRI_STRIPS change v2: Fix '- nr' typo noticed by Marius. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> [v1] URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fd97a055083a8927d1f6458b9015a202f43f8f01 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:23:44 2015 -0700 t_dd_dmatmp: Require HAVE_TRI_STRIPS Two drivers use this file, and both support triangle strips. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=22b73f3c2a09375c49bd6202dabdbff368351315 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:19:44 2015 -0700 t_dd_dmatmp: Require HAVE_TRIANGLES Two drivers use this file, and both support triangles. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=dcd8e49962b550f8854fff20e0369dae5550f640 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:14:08 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_LINE_STRIPS change No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1ecdf956ac699aee5e8c62887a40608accb0ac94 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:08:40 2015 -0700 t_dd_dmatmp: Require HAVE_LINE_STRIPS Two drivers use this file, and both support line strips. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1ab8a69a3b7e2846a602f14aa6e4dd5994be1615 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 14:51:46 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_LINES change No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b8461e03f01167a8bafe7fa9f12ff0ec246533b4 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 12:46:21 2015 -0700 t_dd_dmatmp: Require HAVE_LINES Two drivers use this file, and both support lines. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=265624c5af6f7eaa18bc81cc8aa8aeb0f5801861 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 12:41:33 2015 -0700 t_dd_dmatmp: Indentation and formatting fixes after HAVE_QUADS change No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4ecc387a935d2364c5be02cbf7a72b0a05fa419a Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 12:38:19 2015 -0700 t_dd_dmatmp: Remove HAVE_QUADS support Two drivers use this file, and neither supports quads. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=249ba09f59af870dc55eacede5b1e164873c397d Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 12:36:33 2015 -0700 t_dd_dmatmp: Remove HAVE_QUAD_STRIPS support Two drivers use this file, and neither supports quad strips. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=25543d8ec506ef32599af6f5e0dd735e01b39909 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 11:59:22 2015 -0700 t_dd_dmatmp: Use addition instead of subtraction in loop bounds This is used everywhere else in this file because it avoids problems when count is zero (due to trimming). No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38109 Reviewed-by: Brian Paul <bri...@vmware.com> Cc: Marius Predut <marius.pre...@intel.com> Cc: "10.6 11.0" <mesa-sta...@lists.freedesktop.org> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c0b3b2f7603eab210acdb2e654e5411fe912ca34 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 11:56:20 2015 -0700 t_dd_dmatmp: Pull out common 'count -= count & 3' code This was missing in the HAVE_TRIANGLES path, and that could cause incorrect rendering. No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38109 Reviewed-by: Brian Paul <bri...@vmware.com> Cc: Marius Predut <marius.pre...@intel.com> Cc: "10.6 11.0" <mesa-sta...@lists.freedesktop.org> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=0d475ee2b989ac1697720ca391913e9158156bdc Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 11:50:28 2015 -0700 t_dd_dmatmp: Use '& 3' instead of '% 4' everywhere No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> Cc: "10.6 11.0" <mesa-sta...@lists.freedesktop.org> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fad8d54de7e7f908cb0d06f0b54af8440e689928 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 11:46:50 2015 -0700 t_dd_dmatmp: Clean up improper code formatting from previous patch No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> Cc: "10.6 11.0" <mesa-sta...@lists.freedesktop.org> URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d7bf7969b90f66ee614f2d2225f3a821d5396a89 Author: Ian Romanick <ian.d.roman...@intel.com> Date: Mon Sep 14 11:37:12 2015 -0700 t_dd_dmatmp: Make "count" actually be the count The value passed in count previously was "vertex after the last vertex to be processed." Calling that "count" was misleading and kind of mean. Looking at the code, many functions immediately do "count-start" to get back the true count. That's just silly. If it is better for the loops to be 'for (j = start; j < (start + count); j++)', GCC will do that transformation. NOTE: There is some strange formatting left by this patch. That was done to make it more obvious that the before and after code is equivalent. These will be fixed in the next patch. No piglit regressions on i915 (G33) or radeon (Radeon 7500). v2: Fix a remaining (count-start) in render_quad_strip_verts. Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> Reviewed-by: Brian Paul <bri...@vmware.com> [v1] Cc: "10.6 11.0" <mesa-sta...@lists.freedesktop.org> _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-commit