[cfe-commits] [PATCH] Add three new sanitizers depending on ASan.

Richard Smith richard at metafoo.co.uk
Thu Nov 29 12:37:39 PST 2012



================
Comment at: docs/UsersManual.html:903
@@ -900,3 +902,3 @@
 
-The following more fine-grained checks are also available:
+The following more fine-grained undefined behavior checks are available:
 
----------------
-fsanitize=unsigned-integer-overflow isn't an undefined behavior check. Maybe there's a better way of organizing this section.

================
Comment at: lib/CodeGen/BackendUtil.cpp:173-175
@@ +172,5 @@
+                                      PassManagerBase &PM) {
+  const PassManagerBuilderWrapper *BuilderWrapper =
+      static_cast<const PassManagerBuilderWrapper*>(&Builder);
+  assert(BuilderWrapper);
+  const LangOptions &LangOpts = BuilderWrapper->getLangOpts();
----------------
Maybe cast Builder to 'const PassManagerBuilderWrapper &'?


http://llvm-reviews.chandlerc.com/D142



More information about the cfe-commits mailing list