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

[Richard Henderson]

On 4/26/23 01:42, Taylor Simpson wrote:
> The following items in the CPUHexagonState are 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.
>
> The following items are moved
>      dczero_addr
>      branch_taken
>      this_PC
>
> Suggested-by: Richard Henderson<richard.henderson@linaro.org>
> Signed-off-by: Taylor Simpson<tsimpson@quicinc.com>
> ---
>   target/hexagon/cpu.h       |  3 ---
>   target/hexagon/helper.h    |  2 +-
>   target/hexagon/macros.h    |  6 +++++-
>   target/hexagon/translate.h |  5 ++---
>   target/hexagon/genptr.c    |  6 +++---
>   target/hexagon/op_helper.c |  5 ++---
>   target/hexagon/translate.c | 23 +++++++----------------
>   target/hexagon/README      |  2 +-
>   8 files changed, 21 insertions(+), 31 deletions(-)

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


r~