From 8e35ec06201e120088e08ef23dbd421e2ba5f643 Mon Sep 17 00:00:00 2001 From: "Dr. Chat" Date: Sat, 16 May 2015 13:31:42 -0500 Subject: [PATCH] Rewrite FMA mul sub/add to avoid register shuffling Also downgrade some AVX opcodes to SSE --- src/xenia/cpu/backend/x64/x64_sequences.cc | 106 +++++++++++---------- 1 file changed, 54 insertions(+), 52 deletions(-) diff --git a/src/xenia/cpu/backend/x64/x64_sequences.cc b/src/xenia/cpu/backend/x64/x64_sequences.cc index 48271dca6..51954bb9d 100644 --- a/src/xenia/cpu/backend/x64/x64_sequences.cc +++ b/src/xenia/cpu/backend/x64/x64_sequences.cc @@ -3742,27 +3742,30 @@ EMITTER_OPCODE_TABLE( // TODO(benvanik): use other forms (132/213/etc) to avoid register shuffling. // dest could be src2 or src3 - need to ensure it's not before overwriting dest // perhaps use other 132/213/etc +// Forms: +// - 132 -> $1 = $1 * $3 + $2 +// - 213 -> $1 = $2 * $1 + $3 +// - 231 -> $1 = $2 * $3 + $1 EMITTER(MUL_ADD_F32, MATCH(I, F32<>, F32<>, F32<>>)) { static void Emit(X64Emitter& e, const EmitArgType& i) { // FMA extension if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmadd213ss(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmadd213ss(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmadd231ss(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovss(i.dest, i.src1); - e.vfmadd213ss(i.dest, i.src2, i.src3); - } else { - e.vmovss(e.xmm0, i.src1); - e.vfmadd213ss(e.xmm0, i.src2, i.src3); - e.vmovss(i.dest, e.xmm0); - } + // Dest not equal to anything + e.movss(i.dest, i.src1); + e.vfmadd213ss(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it. Xmm src3 = i.src3; if (i.dest == i.src3) { - e.vmovss(e.xmm0, i.src3); + e.movss(e.xmm0, i.src3); src3 = e.xmm0; } @@ -3777,21 +3780,20 @@ EMITTER(MUL_ADD_F64, MATCH(I, F64<>, F64<>, F64<>>)) { if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmadd213sd(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmadd213sd(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmadd231sd(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovsd(i.dest, i.src1); - e.vfmadd213sd(i.dest, i.src2, i.src3); - } else { - e.vmovsd(e.xmm0, i.src1); - e.vfmadd213sd(e.xmm0, i.src2, i.src3); - e.vmovsd(i.dest, e.xmm0); - } + // Dest not equal to anything + e.movsd(i.dest, i.src1); + e.vfmadd213sd(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it. Xmm src3 = i.src3; if (i.dest == i.src3) { - e.vmovsd(e.xmm0, i.src3); + e.movsd(e.xmm0, i.src3); src3 = e.xmm0; } @@ -3806,15 +3808,14 @@ EMITTER(MUL_ADD_V128, MATCH(I, V128<>, V128<>, V128<>>)) if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmadd213ps(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmadd213ps(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmadd231ps(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovdqa(i.dest, i.src1); - e.vfmadd213ps(i.dest, i.src2, i.src3); - } else { - e.vmovdqa(e.xmm0, i.src1); - e.vfmadd213ps(e.xmm0, i.src2, i.src3); - e.vmovdqa(i.dest, e.xmm0); - } + // Dest not equal to anything + e.vmovqda(i.dest, i.src1); + e.vfmadd213ps(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it. @@ -3844,27 +3845,30 @@ EMITTER_OPCODE_TABLE( // TODO(benvanik): use other forms (132/213/etc) to avoid register shuffling. // dest could be src2 or src3 - need to ensure it's not before overwriting dest // perhaps use other 132/213/etc +// Forms: +// - 132 -> $1 = $1 * $3 - $2 +// - 213 -> $1 = $2 * $1 - $3 +// - 231 -> $1 = $2 * $3 - $1 EMITTER(MUL_SUB_F32, MATCH(I, F32<>, F32<>, F32<>>)) { static void Emit(X64Emitter& e, const EmitArgType& i) { // FMA extension if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmsub213ss(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmsub213ss(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmsub231ss(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovss(i.dest, i.src1); - e.vfmsub213ss(i.dest, i.src2, i.src3); - } else { - e.vmovss(e.xmm0, i.src1); - e.vfmsub213ss(e.xmm0, i.src2, i.src3); - e.vmovss(i.dest, e.xmm0); - } + // Dest not equal to anything + e.movss(i.dest, i.src1); + e.vfmsub213ss(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it. Xmm src3 = i.src3; if (i.dest == i.src3) { - e.vmovss(e.xmm0, i.src3); + e.movss(e.xmm0, i.src3); src3 = e.xmm0; } @@ -3879,21 +3883,20 @@ EMITTER(MUL_SUB_F64, MATCH(I, F64<>, F64<>, F64<>>)) { if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmsub213sd(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmsub213sd(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmsub231sd(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovsd(i.dest, i.src1); - e.vfmsub213sd(i.dest, i.src2, i.src3); - } else { - e.vmovsd(e.xmm0, i.src1); - e.vfmsub213sd(e.xmm0, i.src2, i.src3); - e.vmovsd(i.dest, e.xmm0); - } + // Dest not equal to anything + e.movsd(i.dest, i.src1); + e.vfmsub213sd(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it. Xmm src3 = i.src3; if (i.dest == i.src3) { - e.vmovsd(e.xmm0, i.src3); + e.movsd(e.xmm0, i.src3); src3 = e.xmm0; } @@ -3908,15 +3911,14 @@ EMITTER(MUL_SUB_V128, MATCH(I, V128<>, V128<>, V128<>>)) if (e.IsFeatureEnabled(kX64EmitFMA)) { if (i.dest == i.src1) { e.vfmsub213ps(i.dest, i.src2, i.src3); + } else if (i.dest == i.src2) { + e.vfmsub213ps(i.dest, i.src1, i.src3); + } else if (i.dest == i.src3) { + e.vfmsub231ps(i.dest, i.src1, i.src2); } else { - if (i.dest != i.src2 && i.dest != i.src3) { - e.vmovdqa(i.dest, i.src1); - e.vfmsub213ps(i.dest, i.src2, i.src3); - } else { - e.vmovdqa(e.xmm0, i.src1); - e.vfmsub213ps(e.xmm0, i.src2, i.src3); - e.vmovdqa(i.dest, e.xmm0); - } + // Dest not equal to anything + e.vmovdqa(i.dest, i.src1); + e.vfmsub213ps(i.dest, i.src2, i.src3); } } else { // If i.dest == i.src3, back up i.src3 so we don't overwrite it.