[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