fabi/Xenia-Canary
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[CPU/PPC] Add Linux workaround for fdiv instruction format

Browse Source 
On Linux, these instructions appear to use A format (FRA, FRB, FRT)
instead of the standard X format (RA, RB, RT). Need to verify if this
is needed on Windows or is Linux specific.
This commit is contained in:
Herman S.
2025-10-19 14:08:36 +09:00
parent 52297ea8f8
commit b7add3e0ec
1 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/xenia/cpu/ppc/ppc_emit_fpu.cc
View File

@@ -54,18 +54,35 @@ int InstrEmit_faddsx(PPCHIRBuilder& f, const InstrData& i) {
int InstrEmit_fdivx(PPCHIRBuilder& f, const InstrData& i) {
// frD <- frA / frB
#if XE_PLATFORM_LINUX
// TODO(has207): verify if this is needed on Windows
// On Linux, fdiv needs to use A format fields instead of X format
Value* v = f.Div(f.LoadFPR(i.A.FRA), f.LoadFPR(i.A.FRB));
f.StoreFPR(i.A.FRT, v);
f.UpdateFPSCR(v, i.A.Rc);
#else
Value* v = f.Div(f.LoadFPR(i.X.RA), f.LoadFPR(i.X.RB));
f.StoreFPR(i.X.RT, v);
f.UpdateFPSCR(v, i.X.Rc);
#endif
return 0;
}
int InstrEmit_fdivsx(PPCHIRBuilder& f, const InstrData& i) {
// frD <- frA / frB
#if XE_PLATFORM_LINUX
// TODO(has207): verify if this is needed on Windows
// On Linux, fdivs needs to use A format fields instead of X format
Value* v = f.Div(f.LoadFPR(i.A.FRA), f.LoadFPR(i.A.FRB));
v = f.ToSingle(v);
f.StoreFPR(i.A.FRT, v);
f.UpdateFPSCR(v, i.A.Rc);
#else
Value* v = f.Div(f.LoadFPR(i.X.RA), f.LoadFPR(i.X.RB));
v = f.ToSingle(v);
f.StoreFPR(i.X.RT, v);
f.UpdateFPSCR(v, i.X.Rc);
#endif
return 0;
}