[PATCH] D40668: [Blocks] Inherit sanitizer options from parent decl

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 7 18:43:41 PST 2017


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





More information about the cfe-commits mailing list