[cfe-commits] [PATCH] Refactor -fsanitize arguments parsing: provide a more careful diagnostic.

Richard Smith richard at metafoo.co.uk
Tue Nov 27 22:33:05 PST 2012



================
Comment at: lib/Driver/SanitizerArgs.h:137-140
@@ +136,6 @@
+         I != E; ++I) {
+      unsigned Add, Remove;
+      if (parse(D, Args, *I, Add, Remove, false) &&
+          (Add & Kind))
+        return describeSanitizeArg(Args, *I, Kind);
+    }
----------------
The contract for this function would be nicer if you could just pass in (say) NeedsUbsanRt, not Kind & NeedsUbsanRt.

Adding "Kind &= ~Remove;" here should have that effect, I think.


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

BRANCH
  svn

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list