public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@amd.com>
To: Sam Edwards <cfsworks@gmail.com>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>,
	u-boot@lists.denx.de
Cc: "Alper Nebi Yasak" <alpernebiyasak@gmail.com>,
	"Andrew Scull" <ascull@google.com>,
	"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
	"Kever Yang" <kever.yang@rock-chips.com>,
	"Marek Behún" <kabel@kernel.org>,
	"Nathan Barrett-Morrison" <nathan.morrison@timesys.com>,
	"Pali Rohár" <pali@kernel.org>, "Peng Fan" <peng.fan@nxp.com>,
	"Philip Oberfichtner" <pro@denx.de>,
	"Philipp Tomsich" <philipp.tomsich@vrull.eu>,
	"Quentin Schulz" <quentin.schulz@theobroma-systems.com>,
	"Simon Glass" <sjg@chromium.org>, "Tom Rini" <trini@konsulko.com>
Subject: Re: [RFC PATCH 00/10] Improve ARM target's support for LLVM toolchain
Date: Mon, 22 May 2023 12:39:04 +0200	[thread overview]
Message-ID: <35cdee04-e94c-0915-85cb-89fb0ea9a6d9@amd.com> (raw)
In-Reply-To: <2355e4a5-1474-5579-2171-8339226db14f@gmail.com>

Hi

On 5/21/23 06:59, Sam Edwards wrote:
> On 5/20/23 22:26, Heinrich Schuchardt wrote:
>> Hello Sam,
> 
> Hi Heinrich! Good to hear from you.
> 
>> I guess the documentation and the CI testing would also have to be adjusted.
> 
> Ah, yeah, those are going to be big things for me to look at when this series 
> starts to mature out of the RFC phase. CI is definitely important so that the 
> hard-won compatibility doesn't just decay away. :)
> 
>> What about non-ARM architectures?
> 
> If there's a groundswell of demand for building U-Boot on LLVM, I'd be willing 
> to collaborate with others on getting the other architectures up to parity with 
> GNU. But since the linker scripts, relocation thunks, sections, and whatnot are 
> all arch-specific, I'm only focusing on ARM for now (which is both the arch I 
> need and one of the more common ones).
> 
> Is there a particular arch you'd like to see next? It seems everything U-Boot 
> supports is supported by LLVM, except for Microblaze, NIOS2, and SH.
> 
>> Could you, please, describe how built with lld so that reviewers can test it.
> 
> I've been building with:
> 
> nice make CC='clang --target=armv7l-none-eabi' \
>    LTO_FINAL_LDFLAGS=-fuse-ld=lld LD=ld.lld OBJCOPY=llvm-objcopy
> 
> ...though mostly at this stage I'm just hoping for folks to confirm that this 
> patchset causes no regressions in their existing GNU environments. (Feedback 
> from LLVM-land would be appreciated nonetheless, though!!!)

Dockerfile in repo as I see is using 3 toolchain categories.
1. llvm deb repo
2. kernel.org
3. others - xtensa/arc

For CI loop you should pretty much provide a way how to get toolchain.
That's why would be good to figure it out and then I am happy to take a look at 
changed you have done for Zynq.
Definitely nice to see this happening and I expect more warnings will be visible 
  and they should be fixed.

Thanks,
Michal



  parent reply	other threads:[~2023-05-22 10:39 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-20 20:55 [RFC PATCH 00/10] Improve ARM target's support for LLVM toolchain Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 01/10] makefile: Fix symbol typo in binary_size_check Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 02/10] arm: set alignment properly for asm funcs Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 03/10] arm: exclude eabi_compat from LTO Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 04/10] arm: add __aeabi_memclr in eabi_compat Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 05/10] arm: add aligned-memory aliases to eabi_compat Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 06/10] arm: discard .gnu.version* sections Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 07/10] arm: efi_loader: discard hash, unwind information Sam Edwards
2023-05-22  7:00   ` Ilias Apalodimas
2023-05-22 19:25     ` Sam Edwards
2023-05-22 20:28       ` Ilias Apalodimas
2023-05-23  5:08       ` Heinrich Schuchardt
2023-05-20 20:55 ` [RFC PATCH 08/10] arm: efi_loader: move .dynamic out of .text in EFI Sam Edwards
2023-05-22  7:30   ` Ilias Apalodimas
2023-05-22  8:10   ` Heinrich Schuchardt
2023-05-22 17:13     ` Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 09/10] arm: discard all .dyn* sections Sam Edwards
2023-05-20 20:55 ` [RFC PATCH 10/10] arm: migrate away from sections.c Sam Edwards
2023-05-23  6:56   ` Ilias Apalodimas
2023-05-21  4:26 ` [RFC PATCH 00/10] Improve ARM target's support for LLVM toolchain Heinrich Schuchardt
2023-05-21  4:59   ` Sam Edwards
2023-05-22  6:52     ` Ilias Apalodimas
2023-05-22 19:37       ` Sam Edwards
2023-05-22 20:15         ` Heinrich Schuchardt
2023-05-23  6:54         ` Ilias Apalodimas
2023-05-22 10:39     ` Michal Simek [this message]
2023-05-22 15:30       ` Tom Rini
2023-05-22 19:59         ` Sam Edwards
2023-05-22 21:13           ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=35cdee04-e94c-0915-85cb-89fb0ea9a6d9@amd.com \
    --to=michal.simek@amd.com \
    --cc=alpernebiyasak@gmail.com \
    --cc=ascull@google.com \
    --cc=cfsworks@gmail.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=kabel@kernel.org \
    --cc=kever.yang@rock-chips.com \
    --cc=nathan.morrison@timesys.com \
    --cc=pali@kernel.org \
    --cc=peng.fan@nxp.com \
    --cc=philipp.tomsich@vrull.eu \
    --cc=pro@denx.de \
    --cc=quentin.schulz@theobroma-systems.com \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox