[PATCH] D7540: DAGCombiner: Don't unnecessarily swap operands in ReassociateOps

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 09:29:38 PST 2016

t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.

Looks reasonable to me (and would even if it made AArch64 slightly worse, that would have to be coincidence). I've no idea why they were swapped in the first place.



More information about the llvm-commits mailing list