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

Fri Nov 5 09:06:36 PDT 2021

ZarkoCA planned changes to this revision.
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.

Thanks for the review, I will follow up with all of your suggestions and update the patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112816