Refactor top-level comma rewriting.
authorPat Gavlin <pagavlin@microsoft.com>
Mon, 11 Jul 2016 22:03:42 +0000 (15:03 -0700)
committerPat Gavlin <pagavlin@microsoft.com>
Tue, 12 Jul 2016 22:19:01 +0000 (15:19 -0700)
commit9f60b6cda6d9437ce7f5a9e430b4ac49f71183de
treef1c652da81c6b634032b855354636c1ca86497e2
parent36efe1f7527ed207b25283832b035a0db0fee2c8
Refactor top-level comma rewriting.

Top-level commas were being handled in three separate parts of
rationalize: TreeTransformRationalization, RewriteSimpleTransforms,
and DoPhase. Furthermore, the transform performed by the latter
was distinct from the transforms performed by the former two parts,
as those parts called out to RewriteTopLevelComma in order to do
the necessary rewrite. This change collects the two transforms
in RewriteTopLevelComma and centralizes the comma handling in
RewriteSimpleTransforms.

With the comma handling centralized in RewriteSimpleTransforms,
TreeTransformRationalization was not doing much of anything, so
it has been removed and RewriteSimpleTransforms renamed to
TreeTransformRationalization.
src/jit/compiler.h
src/jit/rationalize.cpp
src/jit/rationalize.h