Skip to content

arch: Fix widening issues when dst is gpr register on x86_64.#509

Merged
pgoodman merged 1 commit intomasterfrom
fix_x86_64_decoder
Apr 21, 2021
Merged

arch: Fix widening issues when dst is gpr register on x86_64.#509
pgoodman merged 1 commit intomasterfrom
fix_x86_64_decoder

Conversation

@lkorenc
Copy link
Contributor

@lkorenc lkorenc commented Apr 20, 2021

Fix two issues with the x86_64 decoder:

  • R15D was ignored when widening dst registers.
  • R8-15 were not widened due to incorrect renaming - fixed by using XED api.

@lkorenc lkorenc added the x86 Related to x86/x86-64/AMD64 lifting support label Apr 20, 2021
@lkorenc lkorenc requested a review from pgoodman April 20, 2021 21:05
@pgoodman pgoodman merged commit 8b5f739 into master Apr 21, 2021
@pgoodman pgoodman deleted the fix_x86_64_decoder branch April 21, 2021 15:03
ekilmer added a commit to ekilmer/remill that referenced this pull request Jun 9, 2021
* master:
  Fixup cxx-common links (lifting-bits#519)
  Use cxx-common 0.1.4 (lifting-bits#517)
  Use system compiler to build (lifting-bits#516)
  Build with LLVM 12 (lifting-bits#515)
  docs: Update stale links (lifting-bits#514)
  Workaround LLVM hardcoded include directory (lifting-bits#513)
  X86 segment reg override (lifting-bits#512)
  CI: Use v0.1.3 (latest) cxx-common prebuilt libraries (lifting-bits#511)
  Error out if there are fewer than four bytes to decode. (lifting-bits#510)
  arch: Fix widening issues when dst is gpr register on x86_64. (lifting-bits#509)
  Find module in build ptr to offset (lifting-bits#508)
  Fix FCOMPmem and FUCOMPmem (lifting-bits#507)
  Use clang from vcpkg in build script (lifting-bits#506)
  BuildPointerToOffset assumes it makes progress (lifting-bits#505)
  Update instruction category for indirect jump & function return (lifting-bits#503)
  treat all pop to pc as function return (lifting-bits#502)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x86 Related to x86/x86-64/AMD64 lifting support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants