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

Fix FCOMPmem and FUCOMPmem #507

Merged
merged 2 commits into from
Apr 16, 2021

Conversation

DCNick3
Copy link
Contributor

@DCNick3 DCNick3 commented Apr 15, 2021

Closes #504

lib/Arch/X86/Semantics/X87.cpp Outdated Show resolved Hide resolved
lib/Arch/X86/Semantics/X87.cpp Outdated Show resolved Hide resolved
@DCNick3
Copy link
Contributor Author

DCNick3 commented Apr 16, 2021

Does not seem to me that this CI failure is related to my changes...

Intel manual says this about the mismatching leaf register:
image

Seems to me this is environment-specific, probably should be ignored in the test case

@pgoodman pgoodman merged commit 3e31f2c into lifting-bits:master Apr 16, 2021
@DCNick3 DCNick3 deleted the fix-FCOMPmem-and-FUCOMPmem branch April 16, 2021 18:19
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Infinite recursion in the semantics
2 participants