[PATCH] D125160: [InstCombine] Reassociate add sequences to reduce dependency chains

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

What is the motivating case for this canonicalization? This requires more justification than the usual instcombine transform because it is the inverse of what another canonicalization pass (-reassociate) would do.

Note that we do have this transform in the backend with "MachineCombiner", so it should handle cases like the ones shown in the test diffs if that is profitable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125160