[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