Skip to content

Commit

Permalink
Delete outdated workaround (#93525)
Browse files Browse the repository at this point in the history
  • Loading branch information
jkotas authored Oct 16, 2023
1 parent 139f45e commit b9159ab
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions src/coreclr/debug/di/process.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9727,12 +9727,6 @@ void CordbProcess::MarshalManagedEvent(DebuggerIPCEvent * pManagedEvent)
// The event still needs to be Marshaled before being used. (see code:CordbProcess::MarshalManagedEvent)
//
//---------------------------------------------------------------------------------------
#if defined(_MSC_VER) && defined(TARGET_ARM)
// This is a temporary workaround for an ARM specific MS C++ compiler bug (internal LKG build 18.1).
// Branch < if (ptrRemoteManagedEvent == NULL) > was always taken and the function always returned false.
// TODO: It should be removed once the bug is fixed.
#pragma optimize("", off)
#endif
bool CordbProcess::CopyManagedEventFromTarget(
const EXCEPTION_RECORD * pRecord,
DebuggerIPCEvent * pLocalManagedEvent)
Expand Down Expand Up @@ -9779,9 +9773,6 @@ bool CordbProcess::CopyManagedEventFromTarget(

return true;
}
#if defined(_MSC_VER) && defined(TARGET_ARM)
#pragma optimize("", on)
#endif

//---------------------------------------------------------------------------------------
// EnsureClrInstanceIdSet - Ensure we have a CLR Instance ID to debug
Expand Down

0 comments on commit b9159ab

Please sign in to comment.