modocache added a comment.

Thanks again for all the reviews, @GorNishanov! Very much appreciated.



================
Comment at: lib/CodeGen/CGCoroutine.cpp:220
   CGF.EmitBlock(ReadyBlock);
+  CXXTryStmt *TryStmt = nullptr;
+  if (Coro.ExceptionHandler && Kind == AwaitKind::Init) {
----------------
GorNishanov wrote:
> I suggest to check whether await_resume expression is noexcept and omit 
> emission of all of the goo. Most of the time the await_ready for 
> initial_suspend will be noexcept and thus we would not need to emit extra 
> stuff.
> 
Oh shoot, I forgot to do this! I'll take care of this in a follow-up commit 
within the next week.


Repository:
  rC Clang

https://reviews.llvm.org/D45860



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to