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

[Balaram Makam via llvm-commits]

bmakam added a comment.

Thanks for testing this patch out, Tim.

Now that I have lured you into benchmarking :) I will go out on a limb and claim that if you run this patch along with its dependent patch http://reviews.llvm.org/D20030 and enable the feature by turning on the flag "-aarch64-ccmp-disable-triangle-latch" you will see the numbers that will be comparable to the numbers I have shown on this thread. This patch by itself is not very interesting and it will get interesting once http://reviews.llvm.org/D20030 is enabled. All the numbers I have presented in this thread are using this suggested heuristic in combination with http://reviews.llvm.org/D20030 enabled. Appreciate your help.


http://reviews.llvm.org/D21299