Re: [PATCH 19/21] Hexagon (target/hexagon) Move pred_written to DisasContext

[Richard Henderson]

On 4/26/23 01:42, Taylor Simpson wrote:
> The pred_written variable in the CPUHexagonState is only used for
> bookkeeping within the translation of a packet.  With recent changes
> that eliminate the need to free TCGv variables, these make more sense
> to be transient and kept in DisasContext.
>
> Suggested-by: Richard Henderson<richard.henderson@linaro.org>
> Signed-off-by: Taylor Simpson<tsimpson@quicinc.com>
> ---
>   target/hexagon/cpu.h       | 2 --
>   target/hexagon/helper.h    | 2 +-
>   target/hexagon/translate.h | 2 +-
>   target/hexagon/genptr.c    | 2 +-
>   target/hexagon/op_helper.c | 5 +++--
>   target/hexagon/translate.c | 9 ++++-----
>   6 files changed, 10 insertions(+), 12 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~