The --deterministic-vars option names temp vars after the source line number, and phi vars after the node index. These are both more stable than using a per-register incrementing counter.
This should help make diffs smaller when testing changes to translate.py: adding/removing variables will not cause unrelated variables to be renamed, keeping the diff brief.
I didn't have any better ideas for a shorter argument name (if you have any better ideas lmk). I did not add the option to the website, since I believe this is mostly useful for local testing & development. It can be added to a ./run_test.py run with -- --deterministic-vars.
Here's an example diff for before/after 7a467845 ("Avoid unwrapping same-value phis when possible"), both with & without this option: https://gist.github.com/zbanks/a21dabff7e0bf2c25908aaa879a77764 . In this case, using --deterministic-vars makes the output 10% shorter.
Maybe I should have this enabled by default in run_tests.py for all project tests?
The
--deterministic-varsoption namestempvars after the source line number, andphivars after the node index. These are both more stable than using a per-register incrementing counter.This should help make diffs smaller when testing changes to
translate.py: adding/removing variables will not cause unrelated variables to be renamed, keeping the diff brief.I didn't have any better ideas for a shorter argument name (if you have any better ideas lmk). I did not add the option to the website, since I believe this is mostly useful for local testing & development. It can be added to a
./run_test.pyrun with-- --deterministic-vars.Here's an example diff for before/after
7a467845("Avoid unwrapping same-value phis when possible"), both with & without this option: https://gist.github.com/zbanks/a21dabff7e0bf2c25908aaa879a77764 . In this case, using--deterministic-varsmakes the output 10% shorter.Maybe I should have this enabled by default in
run_tests.pyfor all project tests?