Re: [PATCH v2 2/8] target/loongarch: Add a check parameter to the TRANS macro

[Richard Henderson]

On 8/11/23 03:02, Song Gao wrote:
> The default check parmeter is ALL, remove TRANS_64 marco.
>
> Suggested-by: Richard Henderson <richard.henderson@linaro.org>
> Signed-off-by: Song Gao <gaosong@loongson.cn>

If you're going to remove TRANS_64, you should simply drop the patch that added it, and be 
careful about the final patch ordering such that the LA32 cpu is created after your patch 
3 (which adds avail_64()).

Otherwise there will be a bisection point at which LA32 is enabled, but avail_64 is not 
enforced.


r~