2020-09-29 21:21:12

by Nick Desaulniers

[permalink] [raw]
Subject: [PATCH] docs: programming-languages: refresh blurb on clang support

Building the kernel with Clang doesn't rely on third party patches, and
has not for a few years now.

Signed-off-by: Nick Desaulniers <[email protected]>
---
Documentation/process/programming-language.rst | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/Documentation/process/programming-language.rst b/Documentation/process/programming-language.rst
index e5f5f065dc24..63af142d64ec 100644
--- a/Documentation/process/programming-language.rst
+++ b/Documentation/process/programming-language.rst
@@ -6,14 +6,15 @@ Programming Language
The kernel is written in the C programming language [c-language]_.
More precisely, the kernel is typically compiled with ``gcc`` [gcc]_
under ``-std=gnu89`` [gcc-c-dialect-options]_: the GNU dialect of ISO C90
-(including some C99 features).
+(including some C99 features). ``clang`` [clang]_ is also supported, see docs
+docs on :ref:`Building Linux with Clang/LLVM <kbuild_llvm>`.

This dialect contains many extensions to the language [gnu-extensions]_,
and many of them are used within the kernel as a matter of course.

-There is some support for compiling the kernel with ``clang`` [clang]_
-and ``icc`` [icc]_ for several of the architectures, although at the time
-of writing it is not completed, requiring third-party patches.
+There is some support for compiling the kernel with ``icc`` [icc]_ for several
+of the architectures, although at the time of writing it is not completed,
+requiring third-party patches.

Attributes
----------
--
2.28.0.709.gb0816b6eb0-goog


2020-09-29 23:11:59

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] docs: programming-languages: refresh blurb on clang support

On 9/29/20 2:19 PM, Nick Desaulniers wrote:
> Building the kernel with Clang doesn't rely on third party patches, and
> has not for a few years now.
>
> Signed-off-by: Nick Desaulniers <[email protected]>
> ---
> Documentation/process/programming-language.rst | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/process/programming-language.rst b/Documentation/process/programming-language.rst
> index e5f5f065dc24..63af142d64ec 100644
> --- a/Documentation/process/programming-language.rst
> +++ b/Documentation/process/programming-language.rst
> @@ -6,14 +6,15 @@ Programming Language
> The kernel is written in the C programming language [c-language]_.
> More precisely, the kernel is typically compiled with ``gcc`` [gcc]_
> under ``-std=gnu89`` [gcc-c-dialect-options]_: the GNU dialect of ISO C90
> -(including some C99 features).
> +(including some C99 features). ``clang`` [clang]_ is also supported, see docs

Drop duplicated "docs" before here or after here.

checkpatch should have caught that.

> +docs on :ref:`Building Linux with Clang/LLVM <kbuild_llvm>`.
>
> This dialect contains many extensions to the language [gnu-extensions]_,
> and many of them are used within the kernel as a matter of course.
>
> -There is some support for compiling the kernel with ``clang`` [clang]_
> -and ``icc`` [icc]_ for several of the architectures, although at the time
> -of writing it is not completed, requiring third-party patches.
> +There is some support for compiling the kernel with ``icc`` [icc]_ for several
> +of the architectures, although at the time of writing it is not completed,
> +requiring third-party patches.
>
> Attributes
> ----------
>

thanks.
--
~Randy

2020-09-30 16:26:16

by Nathan Chancellor

[permalink] [raw]
Subject: Re: [PATCH] docs: programming-languages: refresh blurb on clang support

On Tue, Sep 29, 2020 at 02:19:35PM -0700, Nick Desaulniers wrote:
> Building the kernel with Clang doesn't rely on third party patches, and
> has not for a few years now.
>
> Signed-off-by: Nick Desaulniers <[email protected]>
> ---

Modulo Randy's nit:

Reviewed-by: Nathan Chancellor <[email protected]>

> Documentation/process/programming-language.rst | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/process/programming-language.rst b/Documentation/process/programming-language.rst
> index e5f5f065dc24..63af142d64ec 100644
> --- a/Documentation/process/programming-language.rst
> +++ b/Documentation/process/programming-language.rst
> @@ -6,14 +6,15 @@ Programming Language
> The kernel is written in the C programming language [c-language]_.
> More precisely, the kernel is typically compiled with ``gcc`` [gcc]_
> under ``-std=gnu89`` [gcc-c-dialect-options]_: the GNU dialect of ISO C90
> -(including some C99 features).
> +(including some C99 features). ``clang`` [clang]_ is also supported, see docs
> +docs on :ref:`Building Linux with Clang/LLVM <kbuild_llvm>`.
>
> This dialect contains many extensions to the language [gnu-extensions]_,
> and many of them are used within the kernel as a matter of course.
>
> -There is some support for compiling the kernel with ``clang`` [clang]_
> -and ``icc`` [icc]_ for several of the architectures, although at the time
> -of writing it is not completed, requiring third-party patches.
> +There is some support for compiling the kernel with ``icc`` [icc]_ for several
> +of the architectures, although at the time of writing it is not completed,
> +requiring third-party patches.
>
> Attributes
> ----------
> --
> 2.28.0.709.gb0816b6eb0-goog
>

2020-10-05 15:48:31

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] docs: programming-languages: refresh blurb on clang support

On Tue, 29 Sep 2020 14:19:35 -0700
Nick Desaulniers <[email protected]> wrote:

> Building the kernel with Clang doesn't rely on third party patches, and
> has not for a few years now.
>
> Signed-off-by: Nick Desaulniers <[email protected]>
> ---
> Documentation/process/programming-language.rst | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)

Applied, thanks. I also took the liberty of removing the duplicated
"docs" pointed out by Randy.

jon