FEX-Emu/FEX
1
0
Fork 0
mirror of https://github.com/FEX-Emu/FEX.git synced 2024-12-14 17:38:47 +00:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

TestHarnessRunner: Ensure we are still reconstructing XMM registers if we don't support AVX

Browse Source 
Also fixes a bug where we were destroying the thread context before
reading the data from it, spooky.
This commit is contained in:
Ryan Houdek 2024-06-28 10:41:03 -07:00
parent f2f90eeb82
commit 98b980f7e3
No known key found for this signature in database
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Source/Tools/TestHarnessRunner/TestHarnessRunner.cpp
View File

@ -316,21 +316,23 @@ int main(int argc, char** argv, char** const envp) {
// Just re-use compare state. It also checks against the expected values in config.
memcpy(&State, &ParentThread->Thread->CurrentFrame->State, sizeof(State));
SignalDelegation->UninstallTLSState(ParentThread);
FEX::HLE::_SyscallHandler->TM.DestroyThread(ParentThread, true);
SyscallHandler.reset();
__uint128_t XMM_Low[FEXCore::Core::CPUState::NUM_XMMS];
if (SupportsAVX) {
///< Reconstruct the XMM registers even if they are in split view, then remerge them.
__uint128_t XMM_Low[FEXCore::Core::CPUState::NUM_XMMS];
__uint128_t YMM_High[FEXCore::Core::CPUState::NUM_XMMS];
CTX->ReconstructXMMRegisters(ParentThread->Thread, XMM_Low, YMM_High);
for (size_t i = 0; i < FEXCore::Core::CPUState::NUM_XMMS; ++i) {
memcpy(&State.xmm.avx.data[i][0], &XMM_Low[i], sizeof(__uint128_t));
memcpy(&State.xmm.avx.data[i][2], &YMM_High[i], sizeof(__uint128_t));
}
} else {
CTX->ReconstructXMMRegisters(ParentThread->Thread, reinterpret_cast<__uint128_t*>(State.xmm.sse.data), nullptr);
}
SignalDelegation->UninstallTLSState(ParentThread);
FEX::HLE::_SyscallHandler->TM.DestroyThread(ParentThread, true);
SyscallHandler.reset();
}
#ifndef _WIN32
else {