diff options
| author | Boris Brezillon <boris.brezillon@collabora.com> | 2020-01-31 10:41:35 +0100 |
|---|---|---|
| committer | Marge Bot <eric+marge@anholt.net> | 2020-02-03 12:53:47 +0000 |
| commit | c7e68d8625e4efcc776b5352de3b16b6951fabf7 (patch) | |
| tree | ace2add409c678a8857835bd3d1fa4a96daf4562 | |
| parent | ef89a52fe58e85402d7652d863da1cc90b378772 (diff) | |
pan/midgard: Don't check 'branch && branch->writeout' twice in mir_schedule_alu()
There's a writeout bool storing the result of this test. Use it instead
of duplicating the test.
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3676>
| -rw-r--r-- | src/panfrost/midgard/midgard_schedule.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/panfrost/midgard/midgard_schedule.c b/src/panfrost/midgard/midgard_schedule.c index 101896de211..7654a046a76 100644 --- a/src/panfrost/midgard/midgard_schedule.c +++ b/src/panfrost/midgard/midgard_schedule.c @@ -914,7 +914,7 @@ mir_schedule_alu( /* If we have a render target reference, schedule a move for it */ - if (branch && branch->writeout && (branch->constants.u32[0] || ctx->is_blend)) { + if (writeout && (branch->constants.u32[0] || ctx->is_blend)) { midgard_instruction mov = v_mov(~0, make_compiler_temp(ctx)); sadd = mem_dup(&mov, sizeof(midgard_instruction)); sadd->unit = UNIT_SADD; |