[PATCH] D132412: [InstCombine] ease use constraint in tryFactorization()

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/add.ll:1759
 
+; x * y + x --> (y + 1) * x
+
----------------
arsenm wrote:
> Do we have a backend combine to undo this already?
Specifically, this is bad for us because it breaks integer mad matching 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132412/new/

https://reviews.llvm.org/D132412