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

Vitaly Buka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 18 15:49:21 PST 2021


vitalybuka requested changes to this revision.
vitalybuka added inline comments.
This revision now requires changes to proceed.


================
Comment at: clang/lib/Basic/SanitizerBlacklist.cpp:15
 #include "clang/Basic/FileManager.h"
+#include "clang/Basic/NoSanitizeList.h"
 #include "clang/Basic/SanitizerSpecialCaseList.h"
----------------
dexonsmith wrote:
> 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.
Oh, right, I didn't noticed that. It needs to be fixed.


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