[PATCH] D96974: [clang][patch] Inclusive language, modify filename SanitizerBlacklist.h to NoSanitizeList.h

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 18 14:48:58 PST 2021


dexonsmith added inline comments.


================
Comment at: clang/lib/Basic/SanitizerBlacklist.cpp:15
 #include "clang/Basic/FileManager.h"
+#include "clang/Basic/NoSanitizeList.h"
 #include "clang/Basic/SanitizerSpecialCaseList.h"
----------------
vitalybuka wrote:
> Own header should go first.
Probably a good idea to move the file `clang/lib/Basic/SanitizerBlacklist.cpp` at the same time as its header `clang/include/clang/Basic/SanitizerBlacklist.h`; then you won't be fighting clang-format on this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96974



More information about the cfe-commits mailing list