FEX-Emu/FEX
1
0
Fork 0
mirror of https://github.com/FEX-Emu/FEX.git synced 2025-02-21 23:31:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Arm64: Minor optimization to bfxil and bfi

Browse Source 
When the destination doesn't alias the source, we can remove a final mov
from both of these operations.

Does some minor code improvement.
This commit is contained in:
Ryan Houdek 2023-10-26 20:25:44 -07:00
parent 4045bfd187
commit 61f22911c7
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
FEXCore/Source/Interface/Core/JIT/Arm64/ALUOps.cpp
View File

@ -1227,6 +1227,11 @@ DEF_OP(Bfi) {
// If Dst and SrcDst match then this turns in to a simple BFI instruction.
bfi(EmitSize, Dst, Src, Op->lsb, Op->Width);
}
else if (Dst != Src) {
// If the destination isn't the source then we can move the DstSrc and insert directly.
mov(EmitSize, Dst, SrcDst);
bfi(EmitSize, Dst, Src, Op->lsb, Op->Width);
}
else {
// Destination didn't match the dst source register.
// TODO: Inefficient until FEX can have RA constraints here.
@ -1256,6 +1261,11 @@ DEF_OP(Bfxil) {
// If Dst and SrcDst match then this turns in to a single instruction.
bfxil(EmitSize, Dst, Src, Op->lsb, Op->Width);
}
else if (Dst != Src) {
// If the destination isn't the source then we can move the DstSrc and insert directly.
mov(EmitSize, Dst, SrcDst);
bfxil(EmitSize, Dst, Src, Op->lsb, Op->Width);
}
else {
// Destination didn't match the dst source register.
// TODO: Inefficient until FEX can have RA constraints here.