[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