[Bug bootstrap/107119] New: Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[Bug bootstrap/107119] New: Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[mkuvyrkov at gcc dot gnu.org]

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

            Bug ID: 107119
           Summary: Bootstrap ICE on 32-bit ARM after
                    r13-2871-g1b74b5cb4e9
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
          Assignee: unassigned at gcc dot gnu.org
          Reporter: mkuvyrkov at gcc dot gnu.org
  Target Milestone: ---

After basepoints/gcc-13-2871-g1b74b5cb4e9:
===
commit 1b74b5cb4e9d7191f298245063a8f9c3a1bbeff4
Author: Jeff Law <jeffreyalaw@gmail.com>
Date:   Sun Sep 25 12:23:59 2022 -0400

   [RFA] Minor improvement to coremark, avoid unconditional jump to return
===

Bootstrap on 32-bit ARM (armv8l-linux-gnueabihf) fails with:

00:04:19 during RTL pass: bbro
00:04:19
/home/tcwg-buildslave/workspace/tcwg_gnu_0/abe/snapshots/gcc.git~master/libgomp/env.c:
In function ‘gomp_get_initial_icv_item’:
00:04:19
/home/tcwg-buildslave/workspace/tcwg_gnu_0/abe/snapshots/gcc.git~master/libgomp/env.c:1964:1:
internal compiler error: in target_gen_simple_return, at
/home/tcwg-buildslave/workspace/tcwg_gnu_0/abe/snapshots/gcc.git~master/gcc/config/arm/arm.md:8998
00:04:19  1964 | }
00:04:19       | ^
00:04:19       
/home/tcwg-buildslave/workspace/tcwg_gnu_0/abe/snapshots/gcc.git~master/gcc/config/arm/arm.md:8998
00:04:19 0x7df80b force_nonfallthru_and_redirect(edge_def*, basic_block_def*,
rtx_def*)
00:04:19        ../../../../../../gcc/gcc/cfgrtl.cc:1711
00:04:19 0x7e3681 fixup_reorder_chain
00:04:19        ../../../../../../gcc/gcc/cfgrtl.cc:4070
00:04:19 0x7e4791 cfg_layout_finalize()
00:04:19        ../../../../../../gcc/gcc/cfgrtl.cc:4565
00:04:19 0x1a80f1b execute
00:04:19        ../../../../../../gcc/gcc/bb-reorder.cc:2663
00:04:19 Please submit a full bug report, with preprocessed source (by using
-freport-bug).
00:04:19 Please include the complete backtrace with any bug report.
00:04:19 See <https://gcc.gnu.org/bugs/> for instructions.

Configure line was:

.../configure --with-gnu-as --with-gnu-ld --disable-libmudflap --enable-lto
--enable-shared --without-included-gettext --enable-nls --with-system-zlib
--disable-sjlj-exceptions --enable-gnu-unique-object --enable-linker-build-id
--disable-libstdcxx-pch --enable-c99 --enable-clocale=gnu
--enable-libstdcxx-debug --enable-long-long --with-cloog=no --with-ppl=no
--with-isl=no --disable-multilib --with-float=hard --with-fpu=neon-fp-armv8
--with-mode=thumb --with-arch=armv8-a --enable-threads=posix --enable-multiarch
--enable-libstdcxx-time=yes --enable-gnu-indirect-function
--enable-checking=yes --enable-bootstrap --enable-languages=c,c++,fortran,to

[Bug bootstrap/107119] Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[mkuvyrkov at gcc dot gnu.org]

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

Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeffreyalaw at gmail dot com,
                   |                            |mkuvyrkov at gcc dot gnu.org

--- Comment #1 from Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> ---
Hi Jeff,

Would you please take a look at this?  Please let me know if you need any
assistance with reproducing this.

I've seen this only on 32-bit ARM; doesn't affect AArch64.

[Bug bootstrap/107119] Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[law at gcc dot gnu.org]

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

Jeffrey A. Law <law at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |law at gcc dot gnu.org

--- Comment #2 from Jeffrey A. Law <law at gcc dot gnu.org> ---
Does it still happen after this:
Author: Jeff Law <jeffreyalaw@gmail.com>
Date:   Tue Sep 27 01:44:38 2022 -0400

    Fix ICEs due to recent jump-to-return optimization

    gcc/
            * cfgrtl.cc (fixup_reorder_chain): Verify that simple_return
            and return are available before trying to use them.

[Bug bootstrap/107119] Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[clyon at gcc dot gnu.org]

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

Christophe Lyon <clyon at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |clyon at gcc dot gnu.org

--- Comment #3 from Christophe Lyon <clyon at gcc dot gnu.org> ---
It seems bootstrap is working on my side.

[Bug bootstrap/107119] Bootstrap ICE on 32-bit ARM after r13-2871-g1b74b5cb4e9

[mkuvyrkov at gcc dot gnu.org]

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

Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> changed:

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

--- Comment #4 from Maxim Kuvyrkov <mkuvyrkov at gcc dot gnu.org> ---
(In reply to Jeffrey A. Law from comment #2)
> Does it still happen after this:
> Author: Jeff Law <jeffreyalaw@gmail.com>
> Date:   Tue Sep 27 01:44:38 2022 -0400
> 
>     Fix ICEs due to recent jump-to-return optimization
>     
>     gcc/
>             * cfgrtl.cc (fixup_reorder_chain): Verify that simple_return
>             and return are available before trying to use them.

After this commit the bootstrap still failed, but something from the below
committed on September 29th fixed it:
ada: Fix checking of Refined_State with nested package renamings (detail)
ada: Improve efficiency of slice-of-component assignment (detail)
ada: Further tweak new expansion of contracts (detail)
ada: Remove duplicated doc comment section (detail)
support -gz=zstd for both linker and assembler (detail)
driver, cppdefault: Unbreak bootstrap on Debian/Ubuntu [PR107059] (detail)
Add OPTIONS_H_EXTRA to GTFILES (detail)
aarch64: Rename AARCH64_ISA architecture-level macros (detail)
aarch64: Rename AARCH64_FL architecture-level macros (detail)
aarch64: Rename AARCH64_FL_FOR_ARCH macros (detail)
aarch64: Add "V" to aarch64-arches.def names (detail)
aarch64: Small config.gcc cleanups (detail)
aarch64: Avoid redundancy in aarch64-cores.def (detail)
aarch64: Remove AARCH64_FL_RCPC8_4 [PR107025] (detail)
aarch64: Fix transitive closure of features (detail)
aarch64: Reorder an entry in aarch64-option-extensions.def (detail)
aarch64: Simplify feature definitions (detail)
aarch64: Simplify generation of .arch strings (detail)
aarch64: Avoid std::string in static data (detail)
aarch64: Tweak constness of option-related data (detail)
aarch64: Make more use of aarch64_feature_flags (detail)
aarch64: Tweak contents of flags_on/off fields (detail)
aarch64: Tweak handling of -mgeneral-regs-only (detail)
aarch64: Remove redundant TARGET_* checks (detail)
data-ref: Fix ranges_maybe_overlap_p test (detail)
place `const volatile' objects in read-only sections (detail)
c++: import/export NTTP objects (detail)
libstdc++: Guard use of new built-in with __has_builtin (detail)
c++: introduce TRAIT_TYPE alongside TRAIT_EXPR (detail)
c++: implement __remove_cv, __remove_reference and __remove_cvref (detail)
OpenACC: whole struct vs. component mappings (PR107028) (detail)