From 430f7e991f169f7b80122c1e9ea578c7cce87a9f Mon Sep 17 00:00:00 2001 From: Erik Faye-Lund Date: Thu, 2 Jan 2025 22:34:45 +0100 Subject: [PATCH] pan/midgard: use macros for mir_prev_op / mir_next_op The benefit of macros here is that they don't care about constness, which is going to be benefitial once we stricten constness a bit here. Reviewed-by: Eric R. Smith Part-of: --- src/panfrost/midgard/compiler.h | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/panfrost/midgard/compiler.h b/src/panfrost/midgard/compiler.h index 06d95bfd4a1..52b96a59b2d 100644 --- a/src/panfrost/midgard/compiler.h +++ b/src/panfrost/midgard/compiler.h @@ -347,17 +347,11 @@ mir_remove_instruction(struct midgard_instruction *ins) list_del(&ins->link); } -static inline midgard_instruction * -mir_prev_op(struct midgard_instruction *ins) -{ - return list_last_entry(&(ins->link), midgard_instruction, link); -} +#define mir_prev_op(ins) \ + list_last_entry(&((ins)->link), midgard_instruction, link) -static inline midgard_instruction * -mir_next_op(struct midgard_instruction *ins) -{ - return list_first_entry(&(ins->link), midgard_instruction, link); -} +#define mir_next_op(ins) \ + list_first_entry(&((ins)->link), midgard_instruction, link) #define mir_foreach_block(ctx, v) \ list_for_each_entry(pan_block, v, &ctx->blocks, link)