Re: [PATCH 4/4] arm64: drop binutils version checks
From: Will Deacon
Date: Fri Apr 11 2025 - 12:04:29 EST
On Tue, Apr 08, 2025 at 03:10:57PM +0200, Arnd Bergmann wrote:
> On Tue, Apr 8, 2025, at 10:46, Will Deacon wrote:
> > Hi Arnd,
> >
> > On Mon, Apr 07, 2025 at 11:41:16AM +0200, Arnd Bergmann wrote:
> >> From: Arnd Bergmann <arnd@xxxxxxxx>
> >>
> >> Now that gcc-8 and binutils-2.30 are the minimum versions, a lot of
> >> the individual feature checks can go away for simplification.
> >>
> >> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> >> ---
> >> arch/arm64/Kconfig | 37 ++-------------------------------
> >> arch/arm64/Makefile | 21 ++-----------------
> >> arch/arm64/include/asm/rwonce.h | 4 ----
> >> arch/arm64/kvm/Kconfig | 1 -
> >> arch/arm64/lib/xor-neon.c | 2 +-
> >> 5 files changed, 5 insertions(+), 60 deletions(-)
> >
> > Since some of these checks are dynamic (i.e. they try passing various
> > options to the tools to see if they barf), have you checked that the
> > minimum supported version of clang implements them all?
>
> I did some randconfig build testing with clang-13/lld-13, since that
> is the oldest supported version, and checked that the options are
> all supported. I'm pretty sure it's been there for a long time before
> that already.
Thanks (especially to Mark!) for checking.
Will