ir3/ra: prevent reusing parent interval of reloaded sources

We would set the `src` flag on the interval of reloaded sources.
However, the interval might be merged with its parent when inserted and
the parent wouldn't have this flag set. This caused the parent interval
to potentially be reused to reload later sources. Fix this by setting
the `src` flag on the top-level interval after insertion.

Signed-off-by: Job Noorman <jnoorman@igalia.com>
Fixes: fa22b0901a ("ir3/ra: Add specialized shared register RA/spilling")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33810>
This commit is contained in:
Job Noorman 2025-02-27 19:45:21 +01:00 committed by Marge Bot
parent 9da1af6eb2
commit 2d540b8074

View file

@ -734,6 +734,10 @@ reload_src(struct ra_ctx *ctx, struct ir3_instruction *instr,
interval->physreg_end = best_reg + size;
interval->needs_reload = true;
ir3_reg_interval_insert(&ctx->reg_ctx, &interval->interval);
while (interval->interval.parent)
interval = ir3_reg_interval_to_ra_interval(interval->interval.parent);
interval->src = true;
}