ykjit / yk

yk packages
https://ykjit.github.io/yk/
Other
29 stars 7 forks source link

Switch JIT IR true/false syntax around. #1271

Closed ltratt closed 3 months ago

ltratt commented 3 months ago

Previously the syntax was guard ..., true|false, but because we're about to have some long lines, the true|false part was so far away from guard that I misread which is which. I've previously thought about using RPython's guard_true / guard_false syntax, but the solution is more obvious and requires an even tinier change: switch the syntax to guard true|false, ....

[Another minor bug fixed here: we previously added a spurious space character after "false", which caused me to have some surprising diffs earlier today.]