[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)) {
----------------
samsonov wrote:
> samsonov wrote:
> > 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`
> As an alternative, introduce new SanitizeKind
>   SupportsCoverage = Address | Memory | Leak | Undefined | Integer,
> 
> and replace this if with
>   if (Kinds & SupportsCoverage) { ... }
> 
That is,
  hasOneOf(Sanitizers, SupportsCoverage)

http://reviews.llvm.org/D6849

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