Re: [PATCH] m68k: Fix invalid .section syntax

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ben,

On Mon, Jul 24, 2023 at 7:37 PM Ben Hutchings <ben@xxxxxxxxxxxxxxx> wrote:
On Mon, 2023-07-24 at 14:58 +0200, Geert Uytterhoeven wrote:
On Fri, Jun 16, 2023 at 5:47 PM Ben Hutchings <ben@xxxxxxxxxxxxxxx> wrote:
From: Ben Hutchings <benh@xxxxxxxxxx>

gas supports several different forms for .section for ELF targets,
including:
    .section NAME [, "FLAGS"[, @TYPE[,FLAG_SPECIFIC_ARGUMENTS]]]
and:
    .section "NAME"[, #FLAGS...]

In several places we use a mix of these two forms:
    .section NAME, #FLAGS...

A current development snapshot of binutils (2.40.50.20230611) treats
this mixed syntax as an error.
[...]
I verified there is no difference in the generated object files with my
current toolchain (GNU assembler version 2.38 (m68k-linux-gnu)
using BFD version (GNU Binutils for Ubuntu) 2.38).

Do you think it's worthwhile to fast-track this for v6.5, or is it fine to
queue this for v6.6?

The new version of binutils has been in Debian unstable for over a
month now.  I think this should be fast-tracked.

OK, PR sent.
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux