[PATCH] D97837: [InstCombine] Remove gc.relocate duplicates

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added a comment.

Actually I took a look into original reproducer and. found that picture is more complex on it.
Removing of gc.relocate duplication makes a duplication of phi node. And elimination of duplication of phi node open a room for further simplification of the next gc.relocate.

For original reproducer in the pipeline:
-passes=rewrite-statepoints-for-gc,instcombine,early-cse,instcombine,early-cse,instcombine,early-cse,instcombine,early-cse,instcombine,early-cse
only the last one did not make any simplification, all other did.

So the problem here that there is no pass which does both - simplify gc.relocate and eliminate duplicated phi nodes.


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

https://reviews.llvm.org/D97837