mirrors/dynarmic
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

emit_x64_vector: Simplify "position == 0" case for EmitVectorExtract()

Browse source 
In the event position is zero, we can just treat it as a NOP, given
there's no need to move the data.
This commit is contained in:
Lioncash 2018-09-08 02:02:47 -04:00 • committed by MerryMage
parent 87372917f9
commit f1ebbcd7bc
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/backend/x64/emit_x64_vector.cpp
View file

@ -764,14 +764,17 @@ void EmitX64::EmitVectorExtract(EmitContext& ctx, IR::Inst* inst) {
auto args = ctx.reg_alloc.GetArgumentInfo(inst); auto args = ctx.reg_alloc.GetArgumentInfo(inst);
const Xbyak::Xmm xmm_a = ctx.reg_alloc.UseScratchXmm(args[0]); const Xbyak::Xmm xmm_a = ctx.reg_alloc.UseScratchXmm(args[0]);
const Xbyak::Xmm xmm_b = ctx.reg_alloc.UseScratchXmm(args[1]);
const u8 position = args[2].GetImmediateU8(); const u8 position = args[2].GetImmediateU8();
ASSERT(position % 8 == 0); ASSERT(position % 8 == 0);
code.psrldq(xmm_a, position / 8); if (position != 0) {
code.pslldq(xmm_b, (128 - position) / 8); const Xbyak::Xmm xmm_b = ctx.reg_alloc.UseScratchXmm(args[1]);
code.por(xmm_a, xmm_b);
code.psrldq(xmm_a, position / 8);
code.pslldq(xmm_b, (128 - position) / 8);
code.por(xmm_a, xmm_b);
}
ctx.reg_alloc.DefineValue(inst, xmm_a); ctx.reg_alloc.DefineValue(inst, xmm_a);
} }