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

Modify the windows amd64 unwinder to work as jitted code unwinder on Uni...#259

Merged
janvorli merged 1 commit into
dotnet:masterfrom
janvorli:linux-issue177
Feb 13, 2015
Merged

Modify the windows amd64 unwinder to work as jitted code unwinder on Uni...#259
janvorli merged 1 commit into
dotnet:masterfrom
janvorli:linux-issue177

Conversation

@janvorli

Copy link
Copy Markdown
Member

Update the windows amd64 unwinder from its source to the latest version
and add context pointers updating that was excluded before.
Change methods of the OOPStackUnwinder and OOPStackUnwinderAMD64 to be
static, since the base class has no data members and the unwinding is
always using the processor specific subclass.
Modify the code so that it is built in two flavors. The first one
is the DAC flavor which is used by the debugger. The second one will be
used for jitted code stack frames unwinding on non-Windows systems.

…Unix

Update the windows amd64 unwinder from its source to the latest version
and add context pointers updating that was excluded before.
Change methods of the OOPStackUnwinder and OOPStackUnwinderAMD64 to be
static, since the base class has no data members and the unwinding is
always using the processor specific subclass.
Modify the code so that it is built in two flavors. The first one
is the DAC flavor which is used by the debugger. The second one will be
used for jitted code stack frames unwinding on non-Windows systems.
@janvorli

Copy link
Copy Markdown
Member Author

Represents part of #177

@jkotas

jkotas commented Feb 13, 2015

Copy link
Copy Markdown
Member

LGTM

janvorli added a commit that referenced this pull request Feb 13, 2015
Modify the windows amd64 unwinder to work as jitted code unwinder on Uni...
@janvorli
janvorli merged commit 0e20a9e into dotnet:master Feb 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants