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