[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