[PATCH] D72558: [InstCombine] Fix worklist management when removing guard intrinsic

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

I don't know anything about experimental.guard either, but the change LGTM based on similar transforms.
Instcombine's visit* contract is:

  // Visitation implementation - Implement instruction combining for different
  // instruction types.  The semantics are as follows:
  // Return Value:
  //    null        - No change was made
  //     I          - Change was made, I is still valid, I may be dead though
  //   otherwise    - Change was made, replace I with returned instruction


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72558