[PATCH] D112816: [LLVM][llvm-cov] Inclusive language: rename option -name-whitelist to -name-allowlist

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 2 12:44:59 PDT 2021


vsk added a comment.

Thanks for the patch. We usually go out of our way to not break llvm-cov workflows. I'd suggest:

- Leaving the old -name-whitelist around for a release, but marked cl::Hidden. Using it should print a deprecation warning.
- Adding a llvm-14 release note (llvm/docs/ReleaseNotes.txt) about the planned deprecation and removal of -name-whitelist. llvm-14 would ship both options.
- Posting on llvm-dev (or a forum) about the change.
- Removing -name-whitelist when llvm-15 branches.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112816/new/

https://reviews.llvm.org/D112816



More information about the llvm-commits mailing list