aprantl added inline comments.
================
Comment at: include/clang/Basic/Sanitizers.h:64
- /// \brief Disable all sanitizers.
- void clear() { Mask = 0; }
+ /// \brief Disable the sanitizers specified in \p K (by default, disable
+ /// all sanitizers).
----------------
Might as well delete the `\brief` while you are modifying the comment.
================
Comment at: include/clang/Basic/Sanitizers.h:85
+/// AST generation.
+static inline SanitizerMask getModularSanitizers() {
+ return SanitizerKind::CFI | SanitizerKind::Integer |
----------------
getModularSanitizers is a bit misleading. How about
`getASTTransparentSanitizers` or something along those lines?
https://reviews.llvm.org/D32724
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
