We were not setting all the status flags after the mov operation.
The set_flags method was only setting the C flag because the shift function returns a ArithmeticOpResult with only result and carry.
Now we explicitly set the status flags.
This error is due to bad design of the flags passing mechanism we have in place now and we should
review it to make it better.
We were not setting all the status flags after the mov operation.
The
set_flagsmethod was only setting theCflag because theshiftfunction returns aArithmeticOpResultwith onlyresultandcarry. Now we explicitly set the status flags.This error is due to bad design of the flags passing mechanism we have in place now and we should review it to make it better.