OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

target-i386: Use new deposit and extract ops

Browse Source 
A couple of places where it was easy to identify a right-shift
followed by an extract or and-with-immediate, and the obvious
sign-extract from a high byte register.

Acked-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Richard Henderson <rth@twiddle.net>
This commit is contained in:
Richard Henderson 2016-10-15 11:54:17 -05:00
parent 59a71b4c5b
commit 04fc2f1c8f
1 changed files with 23 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
target/i386/translate.c
View File

@ -383,8 +383,7 @@ static void gen_op_mov_reg_v(TCGMemOp ot, int reg, TCGv t0)
static inline void gen_op_mov_v_reg(TCGMemOp ot, TCGv t0, int reg) static inline void gen_op_mov_v_reg(TCGMemOp ot, TCGv t0, int reg)
{ {
if (ot == MO_8 && byte_reg_is_xH(reg)) { if (ot == MO_8 && byte_reg_is_xH(reg)) {
tcg_gen_shri_tl(t0, cpu_regs[reg - 4], 8); tcg_gen_extract_tl(t0, cpu_regs[reg - 4], 8, 8);
tcg_gen_ext8u_tl(t0, t0);
} else { } else {
tcg_gen_mov_tl(t0, cpu_regs[reg]); tcg_gen_mov_tl(t0, cpu_regs[reg]);
} }
@ -3768,8 +3767,7 @@ static void gen_sse(CPUX86State *env, DisasContext *s, int b,
/* Extract the LEN into a mask. Lengths larger than /* Extract the LEN into a mask. Lengths larger than
operand size get all ones. */ operand size get all ones. */
tcg_gen_shri_tl(cpu_A0, cpu_regs[s->vex_v], 8); tcg_gen_extract_tl(cpu_A0, cpu_regs[s->vex_v], 8, 8);
tcg_gen_ext8u_tl(cpu_A0, cpu_A0);
tcg_gen_movcond_tl(TCG_COND_LEU, cpu_A0, cpu_A0, bound, tcg_gen_movcond_tl(TCG_COND_LEU, cpu_A0, cpu_A0, bound,
cpu_A0, bound); cpu_A0, bound);
tcg_temp_free(bound); tcg_temp_free(bound);
@ -3920,9 +3918,8 @@ static void gen_sse(CPUX86State *env, DisasContext *s, int b,
gen_compute_eflags(s); gen_compute_eflags(s);
} }
carry_in = cpu_tmp0; carry_in = cpu_tmp0;
tcg_gen_shri_tl(carry_in, cpu_cc_src, tcg_gen_extract_tl(carry_in, cpu_cc_src,
ctz32(b == 0x1f6 ? CC_C : CC_O)); ctz32(b == 0x1f6 ? CC_C : CC_O), 1);
tcg_gen_andi_tl(carry_in, carry_in, 1);
} }
switch (ot) { switch (ot) {
@ -5447,6 +5444,9 @@ static target_ulong disas_insn(CPUX86State *env, DisasContext *s,
rm = (modrm & 7) | REX_B(s); rm = (modrm & 7) | REX_B(s);
if (mod == 3) { if (mod == 3) {
if (s_ot == MO_SB && byte_reg_is_xH(rm)) {
tcg_gen_sextract_tl(cpu_T0, cpu_regs[rm - 4], 8, 8);
} else {
gen_op_mov_v_reg(ot, cpu_T0, rm); gen_op_mov_v_reg(ot, cpu_T0, rm);
switch (s_ot) { switch (s_ot) {
case MO_UB: case MO_UB:
@ -5463,6 +5463,7 @@ static target_ulong disas_insn(CPUX86State *env, DisasContext *s,
tcg_gen_ext16s_tl(cpu_T0, cpu_T0); tcg_gen_ext16s_tl(cpu_T0, cpu_T0);
break; break;
} }
}
gen_op_mov_reg_v(d_ot, reg, cpu_T0); gen_op_mov_reg_v(d_ot, reg, cpu_T0);
} else { } else {
gen_lea_modrm(env, s, modrm); gen_lea_modrm(env, s, modrm);