[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] tcg/optimize: Flush data at labels not TCG_OPF_BB_END
From: |
Alex Bennée |
Subject: |
Re: [PATCH 2/2] tcg/optimize: Flush data at labels not TCG_OPF_BB_END |
Date: |
Wed, 21 Oct 2020 15:57:55 +0100 |
User-agent: |
mu4e 1.5.6; emacs 28.0.50 |
Richard Henderson <richard.henderson@linaro.org> writes:
> We can easily propagate temp values through the entire extended
> basic block (in this case, the set of blocks connected by fallthru),
> simply by not discarding the register state at the branch.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
> tcg/optimize.c | 35 ++++++++++++++++++-----------------
> 1 file changed, 18 insertions(+), 17 deletions(-)
>
> diff --git a/tcg/optimize.c b/tcg/optimize.c
> index 220f4601d5..9952c28bdc 100644
> --- a/tcg/optimize.c
> +++ b/tcg/optimize.c
> @@ -1484,29 +1484,30 @@ void tcg_optimize(TCGContext *s)
> }
> }
> }
> - goto do_reset_output;
> + /* fall through */
>
> default:
> do_default:
A random aside:
The optimize function has a lot of goto's in it and I find generally
hard to follow. Anyway:
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
--
Alex Bennée