[PATCH] D87817: [LICM][Coroutine] Don't sink stores to coroutine suspend point.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 18 13:57:36 PDT 2020


efriedma requested changes to this revision.
efriedma added a comment.
This revision now requires changes to proceed.

Okay, I'll wait for the update.



================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:400
+
+        if (auto *SWI = dyn_cast<SwitchInst>(PredBB->getTerminator())) {
+          IntrinsicInst *II = dyn_cast<IntrinsicInst>(SWI->getCondition());
----------------
Is there any particular reason to expect the edge from the coro_suspend to the coro_end is a switch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87817



More information about the llvm-commits mailing list