[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