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

Tue Nov 9 11:21:04 PST 2021

ZarkoCA added a comment.

In D112816#3103937 <https://reviews.llvm.org/D112816#3103937>, @vsk wrote:

> 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.

Again, thanks for the review and apologies if the patch was overzealous in removing the existing option.

I've posted an RFC for the option name change here: https://lists.llvm.org/pipermail/llvm-dev/2021-November/153632.html

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112816