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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 5 03:05:06 PDT 2018


lebedev.ri added a comment.

Having looked through https://reviews.llvm.org/D45842, i find this differential rather more complex.
Maybe instcombine shouldn't be doing this..

It would be interesting to know which of these testcase *aren't* handled by https://reviews.llvm.org/D45842.


Repository:
  rL LLVM

https://reviews.llvm.org/D46336





More information about the llvm-commits mailing list