[PATCH] D34021: [coroutines] Fix co_await for range statement

[Eric Fiselier via Phabricator via cfe-commits]

EricWF updated this revision to Diff 101862.
EricWF added a comment.

- Fix value category of `co_await`/`co_yield` expression. ie use the return type of the resume expression to determine the value category.

@rsmith Should the `CoroutineSuspendExpr` constructor be using `getCallReturnType()` to determine the correct value category, or is the current formulation correct?


https://reviews.llvm.org/D34021

Files:
  include/clang/AST/ExprCXX.h
  include/clang/Sema/Sema.h
  lib/Sema/SemaCoroutine.cpp
  lib/Sema/SemaStmt.cpp
  test/SemaCXX/coawait_range_for.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34021.101862.patch
Type: text/x-patch
Size: 15131 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170608/36697674/attachment-0001.bin>