[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAKwvOdmxkwA7NPSj-bSarurjWc7Vs2vzuT3PnaCJhA00pRWyYA@mail.gmail.com>
Date: Fri, 25 Sep 2020 11:52:44 -0700
From: Nick Desaulniers <ndesaulniers@...gle.com>
To: Florian Fainelli <f.fainelli@...il.com>,
Masahiro Yamada <masahiroy@...nel.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
Nathan Chancellor <natechancellor@...il.com>,
Michal Marek <michal.lkml@...kovi.net>,
Jonathan Corbet <corbet@....net>,
Kees Cook <keescook@...omium.org>,
Sedat Dilek <sedat.dilek@...il.com>,
"open list:CLANG/LLVM BUILD SUPPORT"
<clang-built-linux@...glegroups.com>,
"open list:DOCUMENTATION" <linux-doc@...r.kernel.org>
Subject: Re: [PATCH] Documentation/llvm: Fix clang target examples
On Fri, Sep 25, 2020 at 8:21 AM Florian Fainelli <f.fainelli@...il.com> wrote:
>
> clang --target=<triple> is how we can specify a particular toolchain
> triple to be use, fix the two occurences in the documentation.
Ah right, my mistake. It's either double dash+equals, or single
dash+space. Thanks for the patch. Masahiro, would you mind picking
this up?
Reviewed-by: Nick Desaulniers <ndesaulniers@...gle.com>
>
> Fixes: fcf1b6a35c16 ("Documentation/llvm: add documentation on building w/ Clang/LLVM")
> Signed-off-by: Florian Fainelli <f.fainelli@...il.com>
> ---
> Documentation/kbuild/llvm.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/kbuild/llvm.rst b/Documentation/kbuild/llvm.rst
> index 334df758dce3..dae90c21aed3 100644
> --- a/Documentation/kbuild/llvm.rst
> +++ b/Documentation/kbuild/llvm.rst
> @@ -39,10 +39,10 @@ which can help simplify cross compiling. ::
> ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- make CC=clang
>
> ``CROSS_COMPILE`` is not used to prefix the Clang compiler binary, instead
> -``CROSS_COMPILE`` is used to set a command line flag: ``--target <triple>``. For
> +``CROSS_COMPILE`` is used to set a command line flag: ``--target=<triple>``. For
> example: ::
>
> - clang --target aarch64-linux-gnu foo.c
> + clang --target=aarch64-linux-gnu foo.c
>
> LLVM Utilities
> --------------
> --
> 2.25.1
>
--
Thanks,
~Nick Desaulniers
Powered by blists - more mailing lists