[PATCH] D57722: [Clang][NCF] Sanitizer options: helpers to check if {Kernel, Hardware}ASan is enabled

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 4 16:32:37 PST 2019


vsk added reviewers: eugenis, kcc.
vsk added inline comments.


================
Comment at: clang/include/clang/Basic/Sanitizers.h:89
+  /// * Hardware Kernel ASan
+  bool hasASanOrKASanOrHWASanOrHWKASan() const {
+    return hasASanOrKASan() || hasHWASanOrHWKASan();
----------------
Sorry to bike-shed, but this seems a bit hard to read. How about: 'hasAnyASan'?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57722





More information about the cfe-commits mailing list