translator/x86: Fix rol/ror semantics

falconre / falcon

Binary Analysis Framework in Rust

Apache License 2.0

549 stars 47 forks source link

Closed emmanuel099 closed 4 years ago

emmanuel099 commented 4 years ago

For both, rol and ror, the value assigned to ZF was more than one bit wide
Replaced ZF shifting in rol by truncation (we are interested in the LSB instead of MSB)

endeav0r commented 4 years ago

Im my opinion, you should just commit these straight to master