[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