[PATCH] D92706: [coroutine] should disable inline before calling coro split

Yifeng Dong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 13 21:58:26 PST 2020


dongAxis1944 added inline comments.


================
Comment at: llvm/test/Transforms/Coroutines/coro-inline.ll:1
+; RUN: opt < %s -always-inline -barrier -coro-early -barrier -coro-split -S | FileCheck %s
+; RUN: opt < %s -enable-new-pm  -always-inline -coro-early -coro-split  -S | FileCheck %s
----------------
lxfind wrote:
> aeubanks wrote:
> > Are these `-barrier`s necessary? The test passes without them. And no other test in all of LLVM uses it.
> @dongAxis1944, do you know the context on why this was initially added?
this is because we want to run always_inline, then coro-early, split at last. using barrier, will keep this sequence well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92706



More information about the llvm-commits mailing list