<div dir="ltr">Thanks, yes I've also included changing "whitelist_fun" to "allowlist_fun" in the above patch. <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 9 Nov 2021 at 16:23, Phipps, Alan <<a href="mailto:a-phipps@ti.com">a-phipps@ti.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div class="gmail-m_-2221275460769387079WordSection1">
<p class="MsoNormal">+1 - I made the same change downstream and was looking to upstream it, but you beat me to it.  Will you also do this for the “whitelist_fun” line prefix?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">-Alan<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> <b>On Behalf Of
</b>Zarko Todorovski via llvm-dev<br>
<b>Sent:</b> Tuesday, November 9, 2021 1:17 PM<br>
<b>To:</b> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<b>Subject:</b> [EXTERNAL] [llvm-dev] [RFC] [llvm-cov] Renaming option -name-whitelist to -name-allowlist in llvm-cov<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">Hi all,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I have recently been working on a few inclusive language patches. One such patch:
<a href="https://reviews.llvm.org/D112816" target="_blank">https://reviews.llvm.org/D112816</a> proposes to change the option name for llvm-cov from -name-whitelist to -name-allowlist.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Following up on the reviewer comment: <a href="https://reviews.llvm.org/D112816#3103937" target="_blank">
https://reviews.llvm.org/D112816#3103937</a>, I plan to mark the -name-whitelist as a hidden option with comments in its description stating it will be deprecated and replaced with -name-allowlist in a future release.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Looking for feedback on this proposal.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Zarko<u></u><u></u></p>
</div>
</div>
</div>
</div>

</blockquote></div>