Re: [PATCH v2] kcov: add __no_sanitize_coverage to fix noinstr for all architectures
From: Miguel Ojeda
Date: Thu May 27 2021 - 15:33:28 EST
On Thu, May 27, 2021 at 6:27 PM 'Marco Elver' via Clang Built Linux
<clang-built-linux@xxxxxxxxxxxxxxxx> wrote:
>
> Note: In the Clang case, __has_feature(coverage_sanitizer) is only true
> if the feature is enabled, and therefore we do not require an additional
> defined(CONFIG_KCOV) (like in the GCC case where __has_attribute(..) is
I would put this explanation as a comment.
Other than that:
Reviewed-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
Thanks!
Cheers,
Miguel