[PATCH] D117087: [C++20] [Coroutines] Implement return value optimization for get_return_object

JunMa via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 15 00:01:11 PST 2022


junparser accepted this revision.
junparser added a comment.
This revision is now accepted and ready to land.

LGTM. Thanks!



================
Comment at: clang/lib/CodeGen/CGCoroutine.cpp:650
 
-  if (Stmt *Ret = S.getReturnStmt())
+  if (Stmt *Ret = S.getReturnStmt()) {
+    // Since we already emitted the return value above, so we shouldn't
----------------
can we just remove this?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117087/new/

https://reviews.llvm.org/D117087



More information about the cfe-commits mailing list