[PATCH] D22659: [coroutines] Part 2 of N: Adding Coroutine Intrinsics

Gor Nishanov via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 21:41:08 PDT 2016


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

Documentation updated to state that coro.alloc is required to have heap allocation elision.
Intrinsics attributes tweaked:
coro.alloc is now [], (when it was IntrNoMem, multiple coro.allocs belonging to different coroutines were coalesced into one, breaking the semantics)
coro.begin is now capturing the promise parameter, otherwise, GVN won't recognize operations that may write to the promise.


https://reviews.llvm.org/D22659

Files:
  docs/Coroutines.rst
  include/llvm/Analysis/TargetTransformInfoImpl.h
  include/llvm/IR/Intrinsics.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22659.65471.patch
Type: text/x-patch
Size: 8466 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160726/f22b4213/attachment.bin>


More information about the llvm-commits mailing list