[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