[PATCH] D46336: [InstCombine] Apply binary operator simplifications to associative/commutative cases.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

I think part of this has already landed with:
https://reviews.llvm.org/rL331311

There are proposals trying to do reassociation in a more complete way outside of instcombine:
https://reviews.llvm.org/D45842 (this would catch the cases that I recently added to and-xor-or.ll)
https://reviews.llvm.org/D41574

This is a response to discussions on llvm-dev that instcombine is already trying to do too much. This patch goes against that idea.


Repository:
  rL LLVM

https://reviews.llvm.org/D46336