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

x86-64 doesn't require 3DNow as the README claims #36

Open
fuel-pcbox opened this issue Aug 5, 2024 · 1 comment
Open

x86-64 doesn't require 3DNow as the README claims #36

fuel-pcbox opened this issue Aug 5, 2024 · 1 comment

Comments

@fuel-pcbox
Copy link

No Intel CPU has ever supported 3DNow instructions, and 3DNow isn't required by the x86-64 specifications, but the README claims it is for some reason.

@iximeow
Copy link
Owner

iximeow commented Aug 6, 2024

yeah, that's outright wrong. i'm not sure why i wrote that. 3dnow is accepted by the decoders by default though, even though those instructions presumably #UD on real hardware from the last decade and a half. thank you for filing the issue.

incidentally that means all the helpers to configure decoders for specific microarchitectures are also wrong (since they build on InstDecoder::minimal() and that itself is wrong). so, i'll fix that up at some point this week..

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

No branches or pull requests

2 participants