[PATCH] D23245: [Coroutines] Part 6: Elide dynamic allocation of a coroutine frame when possible

Gor Nishanov via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 10 07:57:14 PDT 2016


GorNishanov updated this revision to Diff 67521.
GorNishanov added a comment.

1. coro.begin now has a parameter pointing directly at coro.alloc if it is present or null, otherwise
2. documentation + test + code updated to support this tweak
3. clarified the FIXME comment explaining that we need to add a safety check when heap allocation elision is possible
4. added a no_elision unit test for the cases when coro.alloc is not provided


https://reviews.llvm.org/D23245

Files:
  docs/Coroutines.rst
  include/llvm/IR/Intrinsics.td
  lib/Transforms/Coroutines/CoroElide.cpp
  lib/Transforms/Coroutines/CoroInstr.h
  lib/Transforms/Coroutines/CoroInternal.h
  lib/Transforms/Coroutines/Coroutines.cpp
  test/Transforms/Coroutines/coro-elide.ll
  test/Transforms/Coroutines/coro-heap-elide.ll
  test/Transforms/Coroutines/restart-trigger.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23245.67521.patch
Type: text/x-patch
Size: 29099 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160810/4286ab88/attachment.bin>


More information about the llvm-commits mailing list