[PATCH] D42606: [Coroutines] Use allocator overload when available
Brian Gesiak via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 14 13:34:46 PST 2018
modocache updated this revision to Diff 134304.
modocache added a comment.
Thanks for the review, @GorNishanov, and for pointing out that part of the standard! I added a reference to the implicit object parameter, as well as some tests for that behavior. And thanks for pointing out the flaw in https://reviews.llvm.org/D41820 -- I'll submit a separate diff to forward the implicit object parameter to promise type constructors.
Repository:
rC Clang
https://reviews.llvm.org/D42606
Files:
lib/Sema/SemaCoroutine.cpp
test/CodeGenCoroutines/coro-alloc.cpp
test/CodeGenCoroutines/coro-gro-nrvo.cpp
test/CodeGenCoroutines/coro-params.cpp
test/SemaCXX/coroutines.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42606.134304.patch
Type: text/x-patch
Size: 16126 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180214/51c8f9ab/attachment-0001.bin>
More information about the cfe-commits
mailing list