EmulateInstructionMIPS64 has been modified to use llvm::MCDisassembler instead of duplicating the decoding logic. Added emulation of few branch instructions for software single stepping.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
Just move stuff from the header to the .cpp file and this is good to go.
source/Plugins/Instruction/MIPS64/EmulateInstructionMIPS64.h | ||
---|---|---|
13–25 | These includes should go into the .cpp file only. | |
80–81 | Move to .cpp | |
114–140 | use "const llvm::MCInst &" so we don't have to know what an llvm::MCInst is in the header file and forward declare llvm::MCInst at top. | |
149–154 | Forward declare all llvm:: types so we don't have to include the headers. |
Minor nit pick: we might want to make the arguments that currently take a "llvm::MCInst &" take "const llvm::MCInst &". Not sure if any function calls use non const functions, so this shouldn't hold up the patch.
source/Plugins/Instruction/MIPS64/EmulateInstructionMIPS64.h | ||
---|---|---|
124–140 | Can we make these all "const llvm::MCInst &" arguments? |
These includes should go into the .cpp file only.