[PATCH] D125160: [InstCombine] Reassociate add sequences to reduce dependency chains
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat May 7 05:31:35 PDT 2022
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
More information about the llvm-commits
mailing list