-
Notifications
You must be signed in to change notification settings - Fork 232
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
Update xed #4202
Comments
I tested the latest xed and none of the three issues are fixed. I feel like we should upstream these bugs and ask xed folks to fix them so that we can benefit from it. Also, one aspect that is making xed complicated is that we need to generate these type info for the intrinsics: https://github.com/Vector35/arch-x86/blob/master/x86_intrinsic_cached_input_types.include. I left comment in https://github.com/Vector35/arch-x86/tree/master/code_generator so I do not need to re-invent the wheels, but it is taking me some time to actually get the patch working again, on top of the latest xed. If possible, I would also love to see a way to dump these types in a more straightforward way. |
Update on these: 1 is believed to be a bug in xed, and I have created an issue for it: intelxed/xed#319 In other words, updating xed itself would not help any these problems right now, so I am giving this issue a lower priority |
I would like to defer the xed update until intelxed/xed#319 is fixed -- which makes sense because otherwise the update does not bring about any known improvements to the x86 support. I kinda expect the xed folks will be able to fix it relatively soon, so the wait would not be too long |
Xed released intelxed/xed@6d87b54 which fixes the "push r16" bug above |
I have update xed to v2024.04.01. But I am caught by surprise that intelxed/xed#319 is indeed NOT fixed. I confused it with intelxed/xed#320, which actually gets fixed. Chances are I will need to update xed again soon to involve the bug fix for intelxed/xed#319 |
I am closing this now since the only issue left within it is #4028 |
It has been a while since we last updated the xed build, and now we are bothered by some xed bugs:
We should check and see if these are fixed in the latest xed build. If not, we should report them upstream
The text was updated successfully, but these errors were encountered: