On 27/2/23 06:41, Richard Henderson wrote:
Tidy up the whole function, hoisting is_bfffo as a common test
for whether tlen and tofs needed. Use tcg_constant_i32, and load
a separate temporary for mask.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  target/m68k/translate.c | 42 ++++++++++++++++++++---------------------
  1 file changed, 20 insertions(+), 22 deletions(-)


          if (ext & 0x800) {
              /* Variable offset */
              tcg_gen_andi_i32(tmp, DREG(ext, 6), 31);
              tcg_gen_rotl_i32(QREG_CC_N, src, tmp);
              tcg_gen_andc_i32(QREG_CC_N, QREG_CC_N, mask);
              tcg_gen_rotr_i32(mask, mask, tmp);
-            if (tofs) {
-                tcg_gen_mov_i32(tofs, tmp);
+            if (is_bfffo) {
+                tofs = tmp;

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>

              }
          } else {

Reply via email to