Reuse flag results in OPCODE_BRANCH_TRUE codegen if the preceding instruction was a comparison that already set the cpu flags
This commit is contained in:
parent
895a0e8703
commit
add3b69649
|
@ -20,7 +20,59 @@ namespace backend {
|
||||||
namespace x64 {
|
namespace x64 {
|
||||||
|
|
||||||
volatile int anchor_control = 0;
|
volatile int anchor_control = 0;
|
||||||
|
template <typename T>
|
||||||
|
static void EmitFusedBranch(X64Emitter& e, const T& i) {
|
||||||
|
bool valid = i.instr->prev && i.instr->prev->dest == i.src1.value;
|
||||||
|
auto opcode = valid ? i.instr->prev->opcode->num : -1;
|
||||||
|
if (valid) {
|
||||||
|
auto name = i.src2.value->name;
|
||||||
|
switch (opcode) {
|
||||||
|
case OPCODE_IS_TRUE:
|
||||||
|
e.jnz(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_IS_FALSE:
|
||||||
|
e.jz(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_EQ:
|
||||||
|
e.je(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_NE:
|
||||||
|
e.jne(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_SLT:
|
||||||
|
e.jl(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_SLE:
|
||||||
|
e.jle(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_SGT:
|
||||||
|
e.jg(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_SGE:
|
||||||
|
e.jge(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_ULT:
|
||||||
|
e.jb(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_ULE:
|
||||||
|
e.jbe(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_UGT:
|
||||||
|
e.ja(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
case OPCODE_COMPARE_UGE:
|
||||||
|
e.jae(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
default:
|
||||||
|
e.test(i.src1, i.src1);
|
||||||
|
e.jnz(name, e.T_NEAR);
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
e.test(i.src1, i.src1);
|
||||||
|
e.jnz(i.src2.value->name, e.T_NEAR);
|
||||||
|
}
|
||||||
|
}
|
||||||
// ============================================================================
|
// ============================================================================
|
||||||
// OPCODE_DEBUG_BREAK
|
// OPCODE_DEBUG_BREAK
|
||||||
// ============================================================================
|
// ============================================================================
|
||||||
|
@ -450,44 +502,58 @@ EMITTER_OPCODE_TABLE(OPCODE_BRANCH, BRANCH);
|
||||||
struct BRANCH_TRUE_I8
|
struct BRANCH_TRUE_I8
|
||||||
: Sequence<BRANCH_TRUE_I8, I<OPCODE_BRANCH_TRUE, VoidOp, I8Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_I8, I<OPCODE_BRANCH_TRUE, VoidOp, I8Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
e.test(i.src1, i.src1);
|
EmitFusedBranch(e, i);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
struct BRANCH_TRUE_I16
|
struct BRANCH_TRUE_I16
|
||||||
: Sequence<BRANCH_TRUE_I16, I<OPCODE_BRANCH_TRUE, VoidOp, I16Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_I16, I<OPCODE_BRANCH_TRUE, VoidOp, I16Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
e.test(i.src1, i.src1);
|
EmitFusedBranch(e, i);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
struct BRANCH_TRUE_I32
|
struct BRANCH_TRUE_I32
|
||||||
: Sequence<BRANCH_TRUE_I32, I<OPCODE_BRANCH_TRUE, VoidOp, I32Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_I32, I<OPCODE_BRANCH_TRUE, VoidOp, I32Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
e.test(i.src1, i.src1);
|
EmitFusedBranch(e, i);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
struct BRANCH_TRUE_I64
|
struct BRANCH_TRUE_I64
|
||||||
: Sequence<BRANCH_TRUE_I64, I<OPCODE_BRANCH_TRUE, VoidOp, I64Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_I64, I<OPCODE_BRANCH_TRUE, VoidOp, I64Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
e.test(i.src1, i.src1);
|
EmitFusedBranch(e, i);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
struct BRANCH_TRUE_F32
|
struct BRANCH_TRUE_F32
|
||||||
: Sequence<BRANCH_TRUE_F32, I<OPCODE_BRANCH_TRUE, VoidOp, F32Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_F32, I<OPCODE_BRANCH_TRUE, VoidOp, F32Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
|
if (i.instr->prev && i.instr->prev->opcode == &OPCODE_IS_TRUE_info &&
|
||||||
|
i.instr->prev->dest == i.src1.value) {
|
||||||
|
e.jnz(i.src2.value->name, e.T_NEAR);
|
||||||
|
} else if (i.instr->prev &&
|
||||||
|
i.instr->prev->opcode == &OPCODE_IS_FALSE_info &&
|
||||||
|
i.instr->prev->dest == i.src1.value) {
|
||||||
|
e.jz(i.src2.value->name, e.T_NEAR);
|
||||||
|
} else {
|
||||||
e.vptest(i.src1, i.src1);
|
e.vptest(i.src1, i.src1);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
e.jnz(i.src2.value->name, e.T_NEAR);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
};
|
};
|
||||||
struct BRANCH_TRUE_F64
|
struct BRANCH_TRUE_F64
|
||||||
: Sequence<BRANCH_TRUE_F64, I<OPCODE_BRANCH_TRUE, VoidOp, F64Op, LabelOp>> {
|
: Sequence<BRANCH_TRUE_F64, I<OPCODE_BRANCH_TRUE, VoidOp, F64Op, LabelOp>> {
|
||||||
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
static void Emit(X64Emitter& e, const EmitArgType& i) {
|
||||||
|
if (i.instr->prev && i.instr->prev->opcode == &OPCODE_IS_TRUE_info &&
|
||||||
|
i.instr->prev->dest == i.src1.value) {
|
||||||
|
e.jnz(i.src2.value->name, e.T_NEAR);
|
||||||
|
} else if (i.instr->prev &&
|
||||||
|
i.instr->prev->opcode == &OPCODE_IS_FALSE_info &&
|
||||||
|
i.instr->prev->dest == i.src1.value) {
|
||||||
|
e.jz(i.src2.value->name, e.T_NEAR);
|
||||||
|
} else {
|
||||||
e.vptest(i.src1, i.src1);
|
e.vptest(i.src1, i.src1);
|
||||||
e.jnz(i.src2.value->name, e.T_NEAR);
|
e.jnz(i.src2.value->name, e.T_NEAR);
|
||||||
}
|
}
|
||||||
|
}
|
||||||
};
|
};
|
||||||
EMITTER_OPCODE_TABLE(OPCODE_BRANCH_TRUE, BRANCH_TRUE_I8, BRANCH_TRUE_I16,
|
EMITTER_OPCODE_TABLE(OPCODE_BRANCH_TRUE, BRANCH_TRUE_I8, BRANCH_TRUE_I16,
|
||||||
BRANCH_TRUE_I32, BRANCH_TRUE_I64, BRANCH_TRUE_F32,
|
BRANCH_TRUE_I32, BRANCH_TRUE_I64, BRANCH_TRUE_F32,
|
||||||
|
|
Loading…
Reference in New Issue