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

[Tom Stellard]

Adding Tim because this breaks an AArch64 test: logical_shifted_reg.ll.  It appears to me the code produced is worse, but I'm not sure.  Here is the output from the test, before is on the left, after is on the right:

http://people.freedesktop.org/~tstellar/aarch_logical_shifted_reg.diff


http://reviews.llvm.org/D7540

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/