[PATCH] D50152: [CodeGen] Merge equivalent block copy/helper functions
Akira Hatanaka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 6 14:06:31 PDT 2018
ahatanak added inline comments.
================
Comment at: lib/CodeGen/CGBlocks.cpp:1643
+ if (Ctx.getBlockVarCopyInits(VD))
+ return true;
+ return false;
----------------
ahatanak wrote:
> rjmccall wrote:
> > Can you just ask Sema to check `canThrow` for the expression and pass it down?
> Since this changes the existing behavior, I made changes to test/CodeGenCXX/block-byref-cxx-objc.cpp to test it. Previously, IRGen would emit an invoke to call `_Block_object_assign` when the constructor was marked as noexcept.
Perhaps I misunderstood your comment, should I have Sema set a flag or something in Expr when it calls a function that can throw?
Repository:
rC Clang
https://reviews.llvm.org/D50152
More information about the cfe-commits
mailing list