[Bug target/104090] New: [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[Bug target/104090] New: [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[sebastian.huber@embedded-brains.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

            Bug ID: 104090
           Summary: [10/11/12 Regression] powerpc: asm machine directive
                    wrong for FSL processors
           Product: gcc
           Version: 10.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: sebastian.huber@embedded-brains.de
  Target Milestone: ---

The latest GCC 10/11/12 branches do not build the powerpc-rtems target due to a
build error in the -mcpu=8540 multilib:

/tmp/sh/b-gcc-powerpc-rtems6/./gcc/xgcc -B/tmp/sh/b-gcc-powerpc-rtems6/./gcc/
-nostdinc -B/tmp/sh/b-gcc-powerpc-rtems6/powerpc-rtems6/m8540/nof/newlib/
-isystem
/tmp/sh/b-gcc-powerpc-rtems6/powerpc-rtems6/m8540/nof/newlib/targ-include
-isystem /home/EB/sebastian_h/src/gcc/newlib/libc/include
-B/tmp/sh/i-powerpc-rtems6/powerpc-rtems6/bin/
-B/tmp/sh/i-powerpc-rtems6/powerpc-rtems6/lib/ -isystem
/tmp/sh/i-powerpc-rtems6/powerpc-rtems6/include -isystem
/tmp/sh/i-powerpc-rtems6/powerpc-rtems6/sys-include  -mcpu=8540 -msoft-float -g
-O2 -O2 -I/home/EB/sebastian_h/src/gcc/libgcc/../newlib/libc/sys/rtems/include
-g -O2 -DIN_GCC  -DCROSS_DIRECTORY_STRUCTURE  -W -Wall -Wno-narrowing
-Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
-Wold-style-definition  -isystem ./include   -g -DIN_LIBGCC2 -fbuilding-libgcc
-fno-stack-protector -Dinhibit_libc  -I. -I. -I../../../.././gcc
-I/home/EB/sebastian_h/src/gcc/libgcc -I/home/EB/sebastian_h/src/gcc/libgcc/.
-I/home/EB/sebastian_h/src/gcc/libgcc/../gcc
-I/home/EB/sebastian_h/src/gcc/libgcc/../include  -DHAVE_CC_TLS  -o _negvdi2.o
-MT _negvdi2.o -MD -MP -MF _negvdi2.dep -DL_negvdi2 -c
/home/EB/sebastian_h/src/gcc/libgcc/libgcc2.c -fvisibility=hidden
-DHIDE_EXPORTS
/tmp/ccZJ18fW.s: Assembler messages:
/tmp/ccZJ18fW.s:24: Error: unrecognized opcode: `isel'
make: *** [Makefile:501: _negvdi2.o] Error 1

The assembler is called like this:

/tmp/sh/b-gcc-powerpc-rtems6/./gcc/as -I
/home/EB/sebastian_h/src/gcc/libgcc/../newlib/libc/sys/rtems/include -I . -I .
-I ../../../.././gcc -I /home/EB/sebastian_h/src/gcc/libgcc -I
/home/EB/sebastian_h/src/gcc/libgcc/. -I
/home/EB/sebastian_h/src/gcc/libgcc/../gcc -I
/home/EB/sebastian_h/src/gcc/libgcc/../include -a32 -me500 -mbig -o _negvdi2.o
_negvdi2.s

Using -me500 seems to be all right, however, the file contains a machine
directive:

    .file    "libgcc2.c"
    .machine ppc
    .section    ".text"

If I remove the ".machine ppc" by hand, the file can be assembled with the
above command line.

The affect of the patch for PR100108 is:

diff -u _negvdi2.s.before _negvdi2.s.after
--- _negvdi2.s.before   2022-01-11 09:07:43.313828636 +0100
+++ _negvdi2.s.after    2022-01-11 08:54:08.424946502 +0100
@@ -1,5 +1,5 @@
        .file   "libgcc2.c"
-       .machine power9
+       .machine ppc
        .section        ".text"
 .Ltext0:
        .align 2

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |10.4

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[sebastian.huber@embedded-brains.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

--- Comment #1 from Sebastian Huber <sebastian.huber@embedded-brains.de> ---
I work on a patch, see:

https://gcc.gnu.org/pipermail/gcc-patches/2022-January/588641.html

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Sebastian Huber <sh@gcc.gnu.org>:

https://gcc.gnu.org/g:1ead972cef7622682c5e6c4fa563c4a894d6e0c3

commit r12-6721-g1ead972cef7622682c5e6c4fa563c4a894d6e0c3
Author: Sebastian Huber <sebastian.huber@embedded-brains.de>
Date:   Tue Jan 18 12:44:53 2022 +0100

    powerc: Fix asm machine directive for some CPUs

    For some CPUs, the assembler machine directive cannot be determined by ISA
    flags.

    gcc/

            PR target/104090
            * config/rs6000/rs6000.cc (rs6000_machine_from_flags): Use also
            rs6000_cpu.

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Sebastian Huber
<sh@gcc.gnu.org>:

https://gcc.gnu.org/g:3cb53c10831be59d967d9dce8e7980fee4703500

commit r11-9534-g3cb53c10831be59d967d9dce8e7980fee4703500
Author: Sebastian Huber <sebastian.huber@embedded-brains.de>
Date:   Tue Jan 18 12:44:53 2022 +0100

    powerpc: Fix asm machine directive for some CPUs

    For some CPUs, the assembler machine directive cannot be determined by ISA
    flags.

    gcc/

            PR target/104090
            * config/rs6000/rs6000.c (rs6000_machine_from_flags): Use also
            rs6000_cpu.

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Sebastian Huber
<sh@gcc.gnu.org>:

https://gcc.gnu.org/g:d568abb25fc799123168aac840372b28bb81f85d

commit r10-10435-gd568abb25fc799123168aac840372b28bb81f85d
Author: Sebastian Huber <sebastian.huber@embedded-brains.de>
Date:   Tue Jan 18 12:44:53 2022 +0100

    powerpc: Fix asm machine directive for some CPUs

    For some CPUs, the assembler machine directive cannot be determined by ISA
    flags.

    gcc/

            PR target/104090
            * config/rs6000/rs6000.c (rs6000_machine_from_flags): Use also
            rs6000_cpu.

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[sebastian.huber@embedded-brains.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

Sebastian Huber <sebastian.huber@embedded-brains.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #5 from Sebastian Huber <sebastian.huber@embedded-brains.de> ---
Fixed on all relevant branches.

[Bug target/104090] [10/11/12 Regression] powerpc: asm machine directive wrong for FSL processors

[judge.packham at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104090

Chris Packham <judge.packham at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |judge.packham at gmail dot com

--- Comment #6 from Chris Packham <judge.packham at gmail dot com> ---
*** Bug 104673 has been marked as a duplicate of this bug. ***