[PATCH] Allow -fsanitize-coverage=N with ubsan, clang part

[Alexey Samsonov]

================
Comment at: lib/Driver/SanitizerArgs.cpp:298
@@ -297,3 +297,3 @@
   // Parse -fsanitize-coverage=N. Currently one of asan/msan/lsan is required.
-  if (NeedsAsan || NeedsMsan || NeedsLsan) {
+  if (NeedsAsan || NeedsMsan || NeedsLsan || NeedsUbsanRt) {
     if (Arg *A = Args.getLastArg(options::OPT_fsanitize_coverage)) {
----------------
NeedsUbsanRt is a compile-time constant. You probably should use
  needsUbsanRt()

(omg, it's terrible). You may consider adding a test that -fsanitize-coverage=1 is reported as unused argument for, e.g., `-fsanitize=dataflow`

http://reviews.llvm.org/D6849

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/