[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