From: Nathan Chancellor <nathan@kernel.org>
To: Paul Cercueil <paul@crapouillou.net>
Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Nick Desaulniers <ndesaulniers@google.com>,
list@opendingux.net, linux-mips@vger.kernel.org,
linux-kernel@vger.kernel.org, llvm@lists.linux.dev
Subject: Re: [PATCH 0/3] MIPS: Add support for LTO
Date: Tue, 14 Dec 2021 11:11:10 -0700 [thread overview]
Message-ID: <YbjePln35WuOX+31@archlinux-ax161> (raw)
In-Reply-To: <20211213224914.1501303-1-paul@crapouillou.net>
On Mon, Dec 13, 2021 at 10:49:11PM +0000, Paul Cercueil wrote:
> Hi,
>
> With this set of patches, it becomes possible to build a LTO'd kernel
> with LLVM/Clang on MIPS.
>
> This was tested on a Ingenic JZ4770 based system. It requires the
> linking step of the vmlinuz.bin to be done with binutils' LD instead of
> LLVM's ld.lld [1], but the vmlinuz ELF itself can be completely built
> with LLVM.
The patches seem fine to me but it might be nice to have this fixed up
before the series is merged so that everything just works when this
configuration is selected. I have added some thoughts on the GitHub
issue.
> Strangely, the LTO'd kernel is bigger in size (3.6 MiB vs. 3.1 MiB
> without LTO), which might be completely normal and I just have wrong
> expectations.
>
> Cheers,
> -Paul
>
> [1]: https://github.com/ClangBuiltLinux/linux/issues/1333
>
> Paul Cercueil (3):
> MIPS: boot/compressed: Disable abicalls
> MIPS: boot/compressed: Build without LTO
> MIPS: Add support for LTO
>
> arch/mips/Kconfig | 1 +
> arch/mips/boot/compressed/Makefile | 7 +++++--
> 2 files changed, 6 insertions(+), 2 deletions(-)
>
> --
> 2.33.0
>
>
next prev parent reply other threads:[~2021-12-14 18:11 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-13 22:49 [PATCH 0/3] MIPS: Add support for LTO Paul Cercueil
2021-12-13 22:49 ` [PATCH 1/3] MIPS: boot/compressed: Disable abicalls Paul Cercueil
2021-12-14 18:06 ` Nathan Chancellor
2021-12-14 19:09 ` Paul Cercueil
2021-12-14 19:38 ` Nick Desaulniers
2021-12-13 22:49 ` [PATCH 2/3] MIPS: boot/compressed: Build without LTO Paul Cercueil
2021-12-14 18:07 ` Nathan Chancellor
2021-12-14 19:43 ` Nick Desaulniers
2021-12-13 22:49 ` [PATCH 3/3] MIPS: Add support for LTO Paul Cercueil
2021-12-14 18:09 ` Nathan Chancellor
2021-12-14 19:45 ` Nick Desaulniers
2021-12-14 18:11 ` Nathan Chancellor [this message]
2021-12-14 19:34 ` [PATCH 0/3] " Nick Desaulniers
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=YbjePln35WuOX+31@archlinux-ax161 \
--to=nathan@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=list@opendingux.net \
--cc=llvm@lists.linux.dev \
--cc=ndesaulniers@google.com \
--cc=paul@crapouillou.net \
--cc=tsbogend@alpha.franken.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