DynamoRIO / DynamoRIO/dynamorio

add extra decoding entries to get the "proper" opcode names for width or reg-reg variants?

Open
#1,307 2 comments 0 reactions 0 assignees View on GitHub
Component-IR Migrated Priority-Low Type-Feature Usability
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 18h
Merged PRs (30d)
30

Description

_From [bruen...@google.com](https://code.google.com/u/109494838902877177630/) on November 01, 2013 10:56:41_

There are a number of opcodes whose mnemonic name changes based on bitwidth or
on whether reg-reg or not: see the sequence in instr.h from OP_cbw through
OP_vmovlhps. These are more substantially different than the condition code
variants: "jae" vs "jnb" doesn't matter much, while "pinsrd" vs "pinsrq" does
matter a litle as it tells you the size of the argment, which is hard to
determine with DR's default disassembly if it's a memory operand. We could add
extra entries that split on modrm or on rex.w or on the data size prefix in
order to have the right name, but it's not clear it's worthwhile.

_Original issue: http://code.google.com/p/dynamorio/issues/detail?id=1307_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.