[PATCH] D33797: [coroutines] Fix rebuilding of dependent coroutine parameters

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 2 16:33:47 PDT 2017


EricWF added inline comments.


================
Comment at: lib/Sema/SemaCoroutine.cpp:1285
+      // We needed to check it, but we don't need to generate code for it.
+      if (!paramDecl->getIdentifier())
+        continue;
----------------
@rsmith Is there a better way to check if the move would be valid for unnamed parameters without building up these expressions?


https://reviews.llvm.org/D33797





More information about the cfe-commits mailing list