linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] docs: programming-languages: refresh blurb on clang support
@ 2020-09-29 21:19 Nick Desaulniers
  2020-09-29 23:10 ` Randy Dunlap
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Nick Desaulniers @ 2020-09-29 21:19 UTC (permalink / raw)
  To: Jonathan Corbet
  Cc: Nick Desaulniers, Nathan Chancellor, linux-doc, linux-kernel,
	clang-built-linux

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 <ndesaulniers@google.com>
---
 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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] docs: programming-languages: refresh blurb on clang support
  2020-09-29 21:19 [PATCH] docs: programming-languages: refresh blurb on clang support Nick Desaulniers
@ 2020-09-29 23:10 ` Randy Dunlap
  2020-09-30 16:21 ` Nathan Chancellor
  2020-10-05 15:45 ` Jonathan Corbet
  2 siblings, 0 replies; 4+ messages in thread
From: Randy Dunlap @ 2020-09-29 23:10 UTC (permalink / raw)
  To: Nick Desaulniers, Jonathan Corbet
  Cc: Nathan Chancellor, linux-doc, linux-kernel, clang-built-linux

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 <ndesaulniers@google.com>
> ---
>  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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] docs: programming-languages: refresh blurb on clang support
  2020-09-29 21:19 [PATCH] docs: programming-languages: refresh blurb on clang support Nick Desaulniers
  2020-09-29 23:10 ` Randy Dunlap
@ 2020-09-30 16:21 ` Nathan Chancellor
  2020-10-05 15:45 ` Jonathan Corbet
  2 siblings, 0 replies; 4+ messages in thread
From: Nathan Chancellor @ 2020-09-30 16:21 UTC (permalink / raw)
  To: Nick Desaulniers
  Cc: Jonathan Corbet, linux-doc, linux-kernel, clang-built-linux

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 <ndesaulniers@google.com>
> ---

Modulo Randy's nit:

Reviewed-by: Nathan Chancellor <natechancellor@gmail.com>

>  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
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] docs: programming-languages: refresh blurb on clang support
  2020-09-29 21:19 [PATCH] docs: programming-languages: refresh blurb on clang support Nick Desaulniers
  2020-09-29 23:10 ` Randy Dunlap
  2020-09-30 16:21 ` Nathan Chancellor
@ 2020-10-05 15:45 ` Jonathan Corbet
  2 siblings, 0 replies; 4+ messages in thread
From: Jonathan Corbet @ 2020-10-05 15:45 UTC (permalink / raw)
  To: Nick Desaulniers
  Cc: Nathan Chancellor, linux-doc, linux-kernel, clang-built-linux

On Tue, 29 Sep 2020 14:19:35 -0700
Nick Desaulniers <ndesaulniers@google.com> 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 <ndesaulniers@google.com>
> ---
>  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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-10-05 15:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-29 21:19 [PATCH] docs: programming-languages: refresh blurb on clang support Nick Desaulniers
2020-09-29 23:10 ` Randy Dunlap
2020-09-30 16:21 ` Nathan Chancellor
2020-10-05 15:45 ` Jonathan Corbet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).