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

2023-08-14 Thread gaosong
在 2023/8/11 下午11:23, 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 Signed-off-by: Song Gao If you're going to remove TRANS_64, you should simply drop the patch that added it, and be careful

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

2023-08-11 Thread 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 Signed-off-by: Song Gao 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

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

2023-08-11 Thread Song Gao
The default check parmeter is ALL, remove TRANS_64 marco. Suggested-by: Richard Henderson Signed-off-by: Song Gao --- target/loongarch/insn_trans/trans_arith.c.inc | 84 +- .../loongarch/insn_trans/trans_atomic.c.inc | 80 +- target/loongarch/insn_trans/trans_bit.c.inc | 56 +- .../lo