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

Backport qemu/qemu@75b208c #1853

Closed
wants to merge 1 commit into from
Closed

Conversation

basavesh
Copy link
Contributor

@basavesh basavesh commented Jul 27, 2023

target/i386: fix operand order for PDEP and PEXT

For PDEP and PEXT, the mask is provided in the memory (mod+r/m)
operand, and therefore is loaded in s->T0 by gen_ldst_modrm.
The source is provided in the second source operand (VEX.vvvv)
and therefore is loaded in s->T1. Fix the order in which
they are passed to the helpers.

@wtdcode
Copy link
Member

wtdcode commented Jul 28, 2023

AFAIK, this is qemu/qemu@75b208c

I will backport this accordingly.

@basavesh
Copy link
Contributor Author

Sounds good to me.

@basavesh
Copy link
Contributor Author

@wtdcode Shall I close this PR as you will be back porting directly? or do you want me to update this PR to match the changes from qemu/qemu@75b208c

@wtdcode
Copy link
Member

wtdcode commented Jul 31, 2023

You could definitely do this (highly welcome!). But please note when you do backport please preserve all origin messages. I did one recently you may refer to: a7a5d18

@basavesh basavesh changed the title FIX the order of arguments for PEXT and PDEP Backport [qemu/qemu@75b208c](https://github.com/qemu/qemu/commit/75b208c28316095c4685e8596ceb9e3f656592e2) Jul 31, 2023
@basavesh basavesh changed the title Backport [qemu/qemu@75b208c](https://github.com/qemu/qemu/commit/75b208c28316095c4685e8596ceb9e3f656592e2) Backport qemu/qemu@75b208c Jul 31, 2023
@basavesh basavesh changed the title Backport qemu/qemu@75b208c Backport [qemu/qemu@75b208c](https://github.com/qemu/qemu/commit/75b208c28316095c4685e8596ceb9e3f656592e2) Aug 1, 2023
@basavesh basavesh changed the title Backport [qemu/qemu@75b208c](https://github.com/qemu/qemu/commit/75b208c28316095c4685e8596ceb9e3f656592e2) Backport #qemu/qemu@75b208c Aug 1, 2023
@basavesh basavesh changed the title Backport #qemu/qemu@75b208c Backport [qemu/qemu@75b208c](link) Aug 1, 2023
@basavesh basavesh changed the title Backport [qemu/qemu@75b208c](link) Backport qemu/qemu@75b208c Aug 1, 2023
@basavesh
Copy link
Contributor Author

basavesh commented Aug 1, 2023

I couldn't get the title correct with the hyperlink. I have made the necessary changes with respect to code and the text.

@wtdcode
Copy link
Member

wtdcode commented Aug 3, 2023

It looks okay but you target the wrong branch. PR always goes to dev.

@basavesh
Copy link
Contributor Author

basavesh commented Aug 3, 2023

Closing this as I created an another PR #1861 for dev branch

@basavesh basavesh closed this Aug 3, 2023
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.

2 participants