pan/mdg: Update r1.w comment
authorAlyssa Rosenzweig <alyssa@collabora.com>
Fri, 4 Jun 2021 21:04:43 +0000 (17:04 -0400)
committerMarge Bot <eric+marge@anholt.net>
Thu, 10 Jun 2021 18:06:10 +0000 (18:06 +0000)
Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11123>

src/panfrost/midgard/midgard_schedule.c

index 35ba87a..e67b0f4 100644 (file)
@@ -1205,12 +1205,9 @@ mir_schedule_alu(
                 predicate.no_cond = true;
         }
 
-        /* When MRT is in use, writeout loops require r1.w to be filled (with a
-         * return address? by symmetry with Bifrost, etc), at least for blend
-         * shaders to work properly. When MRT is not in use (including on SFBD
-         * GPUs), this is not needed. Blend shaders themselves don't know if
-         * they are paired with MRT or not so they always need this, at least
-         * on MFBD GPUs. */
+        /* When MRT is in use, writeout loops require r1.w to be filled with a
+         * return address for the blend shader to jump to.  We always emit the
+         * move for blend shaders themselves for ABI reasons. */
 
         if (writeout && (ctx->inputs->is_blend || ctx->writeout_branch[1])) {
                 vadd = ralloc(ctx, midgard_instruction);