[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.
Tim.
http://reviews.llvm.org/D7540
More information about the llvm-commits
mailing list