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 < revi...@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 > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
