MyJIT Code

Code Merge Request #1: crash when doing 32-bit indexed register move where source and base reg are the same (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

FeepingCreature wants to merge 0 commits from /u/feep/myjit/ to master, 2017-03-20

fix by removing the xor of the target register, since it's unnecessary anyways cause amd64 zero extends 32-bit ops. (https://support.amd.com/TechDocs/24592.pdf section 3.4.5, https://software.intel.com/sites/default/files/managed/a4/60/253665-sdm-vol-1.pdf section 3.4.1.1)

Commit	Date

Discussion

Petr Krajca - 2017-03-20

Status: open --> merged
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Petr Krajca - 2017-03-20

You're right. Thank you for pointing this out and for the bugfix.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.