[PATCH] D21299: [Codegen Prepare] Swap commutative binops before splitting branch condition.

[Balaram Makam via llvm-commits]

bmakam added a comment.

Thanks Renato,

The main motivation for this patch is to fix the regressions we see in spec2006/mcf when http://reviews.llvm.org/D20030 is enabled. I have so far tested this patch in combination with http://reviews.llvm.org/D20030 on spec2006 on Kryo and overall the results are positive(better) for LTO config:

  Benchmark                          Diff
  ---------------------------------------------
  spec2006/astar:ref/BigLakes2048.cfg   0.47%
  spec2006/gcc:ref/g23.in               0.41%
  spec2006/gobmk:ref/nngs.tst          -0.33%
  spec2006/gobmk:ref/score2.tst        -1.14%
  spec2006/gobmk:ref/trevorc.tst       -0.50%
  spec2006/gobmk:ref/trevord.tst       -0.47%
  spec2006/mcf:ref                      8.14%
  spec2006/omnetpp:ref                 -1.75%
  spec2006/perlbench:ref/checkspam.pl  -2.34%
  spec2006/perlbench:ref/splitmail.pl  -1.89%
  spec2006/povray:ref                   1.02%
  spec2006/sjeng:ref                    0.65%
  spec2006/xalancbmk:ref                2.07%


http://reviews.llvm.org/D21299