[PATCH] D103159: [Clang] Enable __has_feature(coverage_sanitizer)

Marco Elver via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 27 09:36:38 PDT 2021


melver added a comment.

In D103159#2784926 <https://reviews.llvm.org/D103159#2784926>, @aaron.ballman wrote:

> In D103159#2784845 <https://reviews.llvm.org/D103159#2784845>, @melver wrote:
>
>> Ping.
>
> FWIW, the usual practice is to ping after no activity on the review for about a week.
>
> That said, LGTM!

Thanks! And sorry for the early Ping. 
We had an earlier version of a patch using the no_sanitize("coverage") attribute in Linux -next and it was causing build robots to complain because they somehow want to use a pre-release of Clang 13 (they probably shouldn't, and also probably not a big deal about them suddenly crashing as they should just upgrade).
In any case, I thought we should get this fixed sooner than later -- and I just sent this: https://lkml.kernel.org/r/20210527162655.3246381-1-elver@google.com


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103159/new/

https://reviews.llvm.org/D103159



More information about the cfe-commits mailing list