[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