-
Notifications
You must be signed in to change notification settings - Fork 2.7k
fix for il to native mapping etw events #15038
Conversation
LGTM |
src/debug/ee/debugger.cpp
Outdated
@@ -3133,7 +3133,7 @@ HRESULT Debugger::GetILToNativeMapping(UINT_PTR pNativeCodeStartAddress, ULONG32 | |||
// | |||
|
|||
HRESULT Debugger::GetILToNativeMappingIntoArrays( | |||
MethodDesc * pMD, | |||
UINT_PTR pCode, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use PCODE
for code addresses?
src/debug/ee/debugger.cpp
Outdated
@@ -3157,7 +3157,7 @@ HRESULT Debugger::GetILToNativeMappingIntoArrays( | |||
|
|||
// Get the JIT info by functionId. | |||
|
|||
DebuggerJitInfo * pDJI = GetLatestJitInfoFromMethodDesc(pMD); | |||
DebuggerJitInfo * pDJI = GetJitInfoFromAddr(pCode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Converting code address back to MethodDesc is not cheap operation. It would be better to use GetJitInfo overload that takes both code address and MethodDesc to avoid this lookup.
I am investigating the test failures and will merge when I understand or fix the arm and mac failures, they seem to be deterministic and I want to understand more. I was sidetracked by other work items but will hopefully have it by tomorrow. |
@dotnet-bot test OSX10.12 x64 Checked Innerloop Build and Test |
@dotnet-bot test Ubuntu armlb Cross Debug Innerloop Build |
Fix for #14947, previously the il to native mapping ETW events would always emit events for the same native code version, this change fixes that.