[PATCH] D97837: [InstCombine] Remove gc.relocate duplicates
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 3 15:38:46 PST 2021
reames added a comment.
Serguei,
At a high level, what's the motivation for this? After thinking about it a bit, this is essentially just CSE. CSE is already done by early-cse and gvn. (In particular, if you run the changed tests through -instcombine -early-cse you get the same result.)
Are you deliberately working around a pass ordering issue here? Or is there something I'm missing with the patch?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97837/new/
https://reviews.llvm.org/D97837
More information about the llvm-commits
mailing list