vsk added a comment.

In https://reviews.llvm.org/D40668#949096, @zaks.anna wrote:

> LGTM.
>
> Thanks!
>
> I was wondering if there are other places where this propagation needs to be 
> added, for example, other calls to GenerateBlockFunction.


Thanks for the review :). Yes there is one other site which calls 
GenerateBlockFunction, but that is done in CodeGenModule, where there are no 
SanOpts to inherit *except* those from the CGM, which is already the default 
behavior. (You can apply suppressions to those by specifying "_block_invoke*" 
in your blacklist.) So I think this is all that's needed.


https://reviews.llvm.org/D40668



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

Reply via email to