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

JunMa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 01:44:42 PDT 2020


junparser created this revision.
junparser added reviewers: lxfind, efriedma, GorNishanov, reames.
junparser added a project: LLVM.
Herald added subscribers: llvm-commits, asbirlea, modocache, hiraditya.
junparser requested review of this revision.

This patch is the refactor of D86190 <https://reviews.llvm.org/D86190>. 
See pr46990(https://bugs.llvm.org/show_bug.cgi?id=46990). Instead of using idea (not sinking store instructions to loop exit blocks which cross coro.suspend intrinsics) in D86190 <https://reviews.llvm.org/D86190>.   This patch split exit block of the suspend point alone and ignore it when do promotion.

Test Plan: check-llvm, pr46990


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D87817

Files:
  llvm/lib/Transforms/Scalar/LICM.cpp
  llvm/test/Transforms/LICM/sink-with-coroutine.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87817.292426.patch
Type: text/x-patch
Size: 4027 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200917/ea2ca683/attachment.bin>


More information about the llvm-commits mailing list