[PATCH] D80445: [EarlyCSE] Common gc.relocate calls.

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Just to be clear, if the indices are the same, gc.relocates would already be removed right?  i.e. the thing you're trying to solve is that the standpoint can have two operands which are the same value and thus two gc.relocates which are equivalent but not textually identical?  Just want to make sure I understand the problem properly.

Another solution to the same problem would be to canonicalize the statepoint in an instcombine rule to avoid duplicate operands in the gc operand section.  Have you explored that possibility?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80445