Re: [PATCH 5/5] Compiler Attributes: make version constraints more consistent

[Nathan Chancellor <nathan@kernel.org>]

On Sat, Aug 03, 2024 at 07:19:33PM +0200, Miguel Ojeda wrote:
> Over time, some version constraints have been added that did not follow
> the formatting of the rest. Fix it for consistency.
> 
> Some of the version requirements can be simplified, too.
> 
> Signed-off-by: Miguel Ojeda <ojeda@kernel.org>

Yeah, I think just having the major version is reasonable, especially
since Clang has moved to GCC's version scheme of x.0 for main, x.1 for
release.

Reviewed-by: Nathan Chancellor <nathan@kernel.org>

> ---
>  include/linux/compiler_attributes.h | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/include/linux/compiler_attributes.h b/include/linux/compiler_attributes.h
> index a6e8c9406f7a..908e59e261c0 100644
> --- a/include/linux/compiler_attributes.h
> +++ b/include/linux/compiler_attributes.h
> @@ -112,7 +112,7 @@
>  
>  /*
>   * Optional: not supported by gcc
> - * Optional: only supported since clang >= 14.0
> + * Optional: only supported since clang >= 14
>   *
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#diagnose_as_builtin
>   */
> @@ -146,7 +146,7 @@
>  #endif
>  
>  /*
> - * Optional: only supported since clang >= 14.0
> + * Optional: only supported since clang >= 14
>   *
>   *   gcc: https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html#index-error-function-attribute
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#error-warning
> @@ -262,7 +262,7 @@
>  #endif
>  
>  /*
> - * Optional: only supported since GCC >= 7.1
> + * Optional: only supported since gcc >= 7
>   *
>   *   gcc: https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html#index-no_005fprofile_005finstrument_005ffunction-function-attribute
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#no-profile-instrument-function
> @@ -280,7 +280,7 @@
>  #define __noreturn                      __attribute__((__noreturn__))
>  
>  /*
> - * Optional: only supported since GCC >= 11.1
> + * Optional: only supported since gcc >= 11
>   *
>   *   gcc: https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html#index-no_005fstack_005fprotector-function-attribute
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#no-stack-protector-safebuffers
> @@ -388,7 +388,7 @@
>  #define __must_check                    __attribute__((__warn_unused_result__))
>  
>  /*
> - * Optional: only supported since clang >= 14.0
> + * Optional: only supported since clang >= 14
>   *
>   *   gcc: https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html#index-warning-function-attribute
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#error-warning
> @@ -400,7 +400,7 @@
>  #endif
>  
>  /*
> - * Optional: only supported since clang >= 14.0
> + * Optional: only supported since clang >= 14
>   *
>   * clang: https://clang.llvm.org/docs/AttributeReference.html#disable-sanitizer-instrumentation
>   *
> -- 
> 2.46.0
>