[PATCH] D70913: [FPEnv][SelectionDAG] Relax chain requirements

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 2 17:19:21 PST 2020


craig.topper added a comment.

I think this doesn't work if a constrained intrinsic is created that doesn't have have any uses, but needs to trigger an exception. The chain output ends up in PendingLoads, but PendingLoads isn't flushed if there's no stores or other operations that need to be ordered. So the basic block ends without connecting the chain from the constrained intrinsic to anything. This makes the node subject to deletion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70913





More information about the llvm-commits mailing list