----- Original Message -----
> Nathan, again thanks. There are a few minor tweaks compared to your
> version:
Thanks for fixing this up!
> - Keeping old layout of "mmix_reg_or_8bit_operand". That looked like
> a spurious change and I prefer the ior construct to the
> if_then_else.
ISTR without this change, there were lots of assembly changes like:
set rx, 6
cmp rz, ry, rx
instead of the previous and better:
cmp rz, ry, 6
(apologies if the assembly syntax isn't correct; hopefully the intent is clear)
but if you double-checked that the assembly didn't change after your changes,
maybe something else that you tweaked fixed this.
> - Replacing undefined-constraint-"H" with "I" instead of removing it.
> It was either renamed early or a genuine typo. Good catch.
Hard not to see it; the gen* machinery complains about undefined constraints. :)
-Nathan
