Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Fix step with stackalloc (#27246) #27351

Merged
merged 1 commit into from
Oct 22, 2019
Merged

Conversation

sdmaclea
Copy link

  • Make ControllerStackInfo::m_returnFrame private

  • Make ControllerStackInfo always capture a return frame

In case the active frame has no managed caller, capture
the unmanaged frame

  • Fix step over stackalloc

Generalize handling of stack allocations and stepping

  • Fix GetFunctionFromToken() argument checking

Check token type is a method before creating a CordbFunction.

Add extra assert to check for invalid tokens

* Make ControllerStackInfo::m_returnFrame private

* Make ControllerStackInfo always capture a return frame

In case the active frame has no managed caller, capture
the unmanaged frame

* Fix step over stackalloc

Generalize handling of stack allocations and stepping

* Fix GetFunctionFromToken() argument checking

Check token type is a method before creating a CordbFunction.

Add extra assert to check for invalid tokens
@sdmaclea sdmaclea added this to the 3.1 milestone Oct 21, 2019
@sdmaclea sdmaclea requested a review from MeiChin-Tsai October 21, 2019 23:24
@sdmaclea sdmaclea self-assigned this Oct 21, 2019
@MeiChin-Tsai
Copy link

approved for 3.1

@sdmaclea sdmaclea merged commit 30583ce into dotnet:release/3.1 Oct 22, 2019
@sdmaclea sdmaclea deleted the stackAlloc3.1 branch October 22, 2019 20:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants