Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extend "x86_mode" to support multiple modes #1550

Closed
derekbruening opened this issue Nov 28, 2014 · 1 comment
Closed

extend "x86_mode" to support multiple modes #1550

derekbruening opened this issue Nov 28, 2014 · 1 comment

Comments

@derekbruening
Copy link
Contributor

From bruen...@google.com on October 08, 2014 00:04:22

Even if for ARM there is no mixed-mode in user-mode and we wouldn't care
about it in regular DR unless we do whole-OS, for drdecodelib we do want to
support multiple modes, or possibly for a tool who wants to go decode a
file.

The proposal is to deprecate {get,set}_x86_mode() and replace them with
{get,set}_isa_mode(dcontext, enum) where the enum can be extended for each
mode: IA-32 vs AMD64, ARM vs Thumb, etc.

We'll move set_x86_mode will be moved to core/arch/x86/decode_private.h and
swap all internal uses to the new routine as well.

Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1550

@derekbruening
Copy link
Contributor Author

From derek.br...@gmail.com on October 21, 2014 09:33:13

This issue was closed by revision r2890 .

Status: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant