panfrost: Remove asserts in panfrost_pack_work_groups_compute
authorAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Thu, 12 Dec 2019 16:30:20 +0000 (11:30 -0500)
committerMarge Bot <eric+marge@anholt.net>
Mon, 16 Dec 2019 19:48:28 +0000 (19:48 +0000)
It's a hot routine and these are exceedingly unlikely to break.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3067>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3067>

src/panfrost/encoder/pan_invocation.c

index ecde3da4648259d5426c8c57008117ac7a7b8967..0ec7c634a24f4765578c8fd4ecdd339b94d8a177 100644 (file)
@@ -50,17 +50,7 @@ panfrost_pack_work_groups_compute(
         unsigned size_z,
         bool quirk_graphics)
 {
-        /* First of all, all 6 values are off-by-one (strictly positive).
-         * Account for that, first by ensuring all values are strictly positive
-         * and then by offsetting */
-
-        assert(num_x > 0);
-        assert(num_y > 0);
-        assert(num_z > 0);
-
-        assert(size_x > 0);
-        assert(size_y > 0);
-        assert(size_z > 0);
+        /* First of all, all 6 values are off-by-one (strictly positive). */
 
         num_x = MALI_POSITIVE(num_x);
         num_y = MALI_POSITIVE(num_y);