[PATCH] D98728: [LICM] Fix a crash when sinking instructions w/token operands

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:1515
       if (Loop *OLoop = LI->getLoopFor(OInst->getParent()))
-        if (!OLoop->contains(&PN)) {
+        if (!OLoop->contains(&PN) && !Op->getType()->isTokenTy()) {
           PHINode *OpPN =
----------------
fhahn wrote:
> It might be helpful for future readers of the code if the token-type check would be separate & with a brief comment (perhaps an early continue). I'm fine either way, please feel free to disregard the comment.
I played with a couple variations, and everything I came up with was harder to read and lengthened the code a lot.  I ended up slightly rewording the comments describing the actions being done and left it at that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98728