[PATCH] D112642: [clang][NFC] Inclusive terms: Replace uses of whitelist in clang/lib/StaticAnalyzer
    Zarko Todorovski via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Oct 29 08:13:43 PDT 2021
    
    
  
ZarkoCA marked 6 inline comments as done.
ZarkoCA added a comment.
In D112642#3093559 <https://reviews.llvm.org/D112642#3093559>, @aaron.ballman wrote:
> Thank you for this! Mostly just bikeshedding on names (allowlist as a verb sounds weird to me), feel free to take or leave the suggestions. LG aside from the formatting nits.
Yes, it didn't seem that right to me either but thought I'd get the ball rolling with something for now. Thanks for the suggestions, it looks much better to me now.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112642/new/
https://reviews.llvm.org/D112642
    
    
More information about the cfe-commits
mailing list