[PATCH] D96203: [clang][patch] Modify sanitizer options names: renaming blacklist to blocklist

[Vitaly Buka via cfe-commits]

To my taste "exclude" better represents meaning than "disallow".
E.g. disallow can be interpreted as "make them fatal". We can fix
2a4317bfb319e for consistency later.
-fsanitize-exclude-list= ?

On Thu, 25 Feb 2021 at 09:54, Melanie Blower via Phabricator <
reviews at reviews.llvm.org> wrote:

> mibintc added a comment.
>
> In D96203#2587942 <https://reviews.llvm.org/D96203#2587942>, @echristo
> wrote:
>
> > Exclusion list? or exclude?
>
> In 2a4317bfb319e <
> https://reviews.llvm.org/rG2a4317bfb319e3ef26bf690e35b00be176bcd37f>,
> (Fangrui Song 2020-06-19) changed option name fsanitize-coverage-whitelist
> to fsanitize-coverage-allowlist.  So another possibility is AllowList and
> DisallowList
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D96203/new/
>
> https://reviews.llvm.org/D96203
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210225/f5349c01/attachment.html>