<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 19, 2020 at 10:43 PM Fangrui Song via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</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">MaskRay added inline comments.<br>
<br>
<br>
================<br>
Comment at: clang/docs/ClangCommandLineReference.rst:891<br>
<br>
 Restrict sanitizer coverage instrumentation exclusively to modules and functions that match the provided special case list, except the blacklisted ones<br>
<br>
----------------<br>
echristo wrote:<br>
> MaskRay wrote:<br>
> > echristo wrote:<br>
> > > MaskRay wrote:<br>
> > > > echristo wrote:<br>
> > > > > I'd remove the uses of blacklist and whitelist here and below. Just have the language and the documentation be for the new option.<br>
> > > > `clang-tblgen -gen-opt-docs` does not seem to provide a feature to hide an option. HelpHidden does not hide the option.<br>
> > > Please remove blacklist and whitelist from the option description.<br>
> > I can do that but the next person updating the documentation will add them back.<br>
> > <br>
> > I believe `clang-tblgen -gen-opt-docs` just doesn't have the feature. Maybe we can delete the old options earlier so that we don't need to worry about the documentation.<br>
> Ugh. Yes, I hope so too. Can you raise it on cfe-dev?<br>
I can but I need to understand more about `clang-tblgen -gen-opt-docs` first...<br>
<br></blockquote><div><br></div><div>I meant deleting the old options :)</div><div><br></div><div>-eric</div></div></div>