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

[Vitaly Buka via Phabricator via cfe-commits]

vitalybuka added a comment.

>> FWIW I would prefer denylist as well. (Also uses of whitelist should be allowlist, but also incremental :)

I feel like existing and proposed value do match the meaning of this list.
maybe ignorelist, skiplist (can be confused with data structure?), or just asan_no_sanitize.txt or even no_asan.txt

> I can change D82244 <https://reviews.llvm.org/D82244> used blocklist to denylist . If there is no need for compatibility, I'll just replace the strings. If there is need for compatibility, I can make blocklist an alias.

I don't think we need compatibility for D82244 <https://reviews.llvm.org/D82244>. If we change clang, we can update D82244 <https://reviews.llvm.org/D82244> to the same for consistency.



================
Comment at: clang/unittests/Driver/SanitizerArgsTest.cpp:134-136
               Contains(StrEq("-fxray-always-instrument=" + XRayWhitelist)));
   EXPECT_THAT(Command.getArguments(),
+              Contains(StrEq("-fxray-never-instrument=" + XRayBlocklist)));
----------------
always/never?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96203