[Bug target/115253] New: New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[Bug target/115253] New: New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[thiago.bauermann at linaro dot org]

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

            Bug ID: 115253
           Summary: New tests added by r14-10122-gad45086178d833 fail on
                    Cortex M23 and M55 bare metal targets
           Product: gcc
           Version: 15.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: thiago.bauermann at linaro dot org
  Target Milestone: ---

Commit g:ad45086178d833254d66fab518b14234418f002b, r14-10122-gad45086178d833,
introduces the tests gcc.target/arm/cmse/extend-param.c and
gcc.target/arm/cmse/extend-return.c.

They have the following failures on --with-cpu=cortex-m23, --with-float=soft,
--target=arm-eabi (full configure flags further below):

Running gcc:gcc.target/arm/cmse/cmse.exp ...
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_boolSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_enumSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_shortSignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_shortUnsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_signSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O0   check-function-bodies
__acle_se_unsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_boolSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_enumSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_shortSignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_shortUnsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_signSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O1   check-function-bodies
__acle_se_unsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_boolSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_enumSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_shortSignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_shortUnsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_signSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O2   check-function-bodies
__acle_se_unsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_boolSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_enumSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_shortSignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_shortUnsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_signSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -O3 -g   check-function-bodies
__acle_se_unsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_boolSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_enumSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_shortSignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_shortUnsignSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_signSecureFunc
FAIL: gcc.target/arm/cmse/extend-param.c -Os   check-function-bodies
__acle_se_unsignSecureFunc
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
boolNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
enumNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
shortSignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
shortUnsignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
signNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0  (internal compiler error: in
extract_insn, at recog.cc:2812)
FAIL: gcc.target/arm/cmse/extend-return.c -O0  (test for excess errors)
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
boolNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
enumNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
shortSignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
shortUnsignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
signNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1  (internal compiler error: in
extract_insn, at recog.cc:2812)
FAIL: gcc.target/arm/cmse/extend-return.c -O1  (test for excess errors)
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
boolNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
enumNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
shortSignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
shortUnsignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
signNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2  (internal compiler error: in
extract_insn, at recog.cc:2812)
FAIL: gcc.target/arm/cmse/extend-return.c -O2  (test for excess errors)
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
boolNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
enumNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
shortSignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
shortUnsignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
signNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g  (internal compiler error: in
extract_insn, at recog.cc:2812)
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g  (test for excess errors)
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
boolNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
enumNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
shortSignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
shortUnsignNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
signNonsecure0
UNRESOLVED: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os  (internal compiler error: in
extract_insn, at recog.cc:2812)
FAIL: gcc.target/arm/cmse/extend-return.c -Os  (test for excess errors)

And these failures on --with-arch=armv8.1-m.main+mve.fp+fp.dp,
--with-float=hard, --target=arm-eabi (Cortex M55):

Running gcc:gcc.target/arm/cmse/cmse.exp ...
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
boolNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
enumNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
shortSignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
shortUnsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
signNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O0   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
boolNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
enumNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
shortSignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
shortUnsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
signNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O1   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
boolNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
enumNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
shortSignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
shortUnsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
signNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O2   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
boolNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
enumNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
shortSignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
shortUnsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
signNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -O3 -g   check-function-bodies
unsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
boolNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
enumNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
shortSignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
shortUnsignNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
signNonsecure0
FAIL: gcc.target/arm/cmse/extend-return.c -Os   check-function-bodies
unsignNonsecure0

Here's how to reproduce with a cross-compiler on x86_64-linux-gnu:

For Cortex M23:

# Create combined tree:
$ mkdir /tmp/combined-tree-src
$ cd ~/src/newlib-cygwin && find . -print | cpio -pdlm /tmp/combined-tree-src
$ cd ~/src/binutils-gdb && find . -print | cpio -pdlmu /tmp/combined-tree-src
$ cd ~/src/gcc && find . -print | cpio -pdlmu /tmp/combined-tree-src
$ mkdir /tmp/build && cd /tmp/build
$ /bin/bash /tmp/combined-tree-src/configure \
    SHELL=/bin/bash \
    --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 \
    --enable-threads=no \
    --disable-multiarch \
    --disable-multilib \
    --with-mode=thumb \
    --with-cpu=cortex-m23 \
    --with-float=soft \
    --with-headers=yes \
    --with-native-system-header-dir=/include \
    --with-newlib \
    --enable-checking=yes \
    --disable-bootstrap \
    --disable-gdb \
    --enable-languages=c,c++,lto \
    --prefix=/tmp/thumb-m23-soft-eabi \
    --build=x86_64-pc-linux-gnu \
    --host=x86_64-pc-linux-gnu \
    --target=arm-eabi
$ make check-c \
    CFLAGS_FOR_BUILD="-pipe" \
    CXXFLAGS_FOR_BUILD="-pipe" \
    -j $(nproc) \
    -w \
    RUNTESTFLAGS=gcc.target/arm/cmse/cmse.exp

For Cortext M55 it's the same as above except with this configure command
instead:

$ /bin/bash /tmp/combined-tree-src/configure \
    SHELL=/bin/bash \
    --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 \
    --enable-threads=no \
    --disable-multiarch \
    --disable-multilib \
    --with-mode=thumb \
    --with-arch=armv8.1-m.main+mve.fp+fp.dp \
    --with-float=hard \
    --with-headers=yes \
    --with-native-system-header-dir=/include \
    --with-newlib \
    --enable-checking=yes \
    --disable-bootstrap \
    --disable-gdb \
    --enable-languages=c,c++,lto \
    --prefix=/tmp/thumb-m55-hard-eabi \
    --build=x86_64-pc-linux-gnu \
    --host=x86_64-pc-linux-gnu \
    --target=arm-eabi

[Bug target/115253] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[thiago.bauermann at linaro dot org]

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

--- Comment #1 from Thiago Jung Bauermann <thiago.bauermann at linaro dot org> ---
Created attachment 58297
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=58297&action=edit
gcc.log for Cortext M23

[Bug target/115253] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[thiago.bauermann at linaro dot org]

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

--- Comment #2 from Thiago Jung Bauermann <thiago.bauermann at linaro dot org> ---
Created attachment 58298
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=58298&action=edit
gcc.log for Cortext M55

These log files were obtained with GCC commit
g:c5a7628470a7fb801ebeea82e16a4549db43bfa5, r15-853-gc5a7628470a7fb.

[Bug target/115253] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[thiago.bauermann at linaro dot org]

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

Thiago Jung Bauermann <thiago.bauermann at linaro dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andre.simoesdiasvieira@arm.
                   |                            |com, Richard.Ball at arm dot com

--- Comment #3 from Thiago Jung Bauermann <thiago.bauermann at linaro dot org> ---
CCing commit authors.

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |14.2
             Target|                            |arm

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[torbjorn.svensson at foss dot st.com]

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

Torbjörn SVENSSON <torbjorn.svensson at foss dot st.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |torbjorn.svensson at foss dot st.c
                   |                            |om

--- Comment #4 from Torbjörn SVENSSON <torbjorn.svensson at foss dot st.com> ---
Patches sent that addresses both the Cortex-M23 and the Cortex-M55 fails.

M23 patch: https://gcc.gnu.org/pipermail/gcc-patches/2024-June/653882.html
M55 patch: https://gcc.gnu.org/pipermail/gcc-patches/2024-June/653884.html

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[azoff at gcc dot gnu.org]

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

Torbjorn SVENSSON <azoff at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at gcc dot gnu.org      |azoff at gcc dot gnu.org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1
                 CC|                            |azoff at gcc dot gnu.org
   Last reconfirmed|                            |2024-06-07

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #5 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Torbjorn Svensson <azoff@gcc.gnu.org>:

https://gcc.gnu.org/g:65bd0655ece268895e5018e393bafb769e201c78

commit r15-1200-g65bd0655ece268895e5018e393bafb769e201c78
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Thu Jun 6 17:12:11 2024 +0200

    arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

    Properly handle zero and sign extension for Armv8-M.baseline as
    Cortex-M23 can have the security extension active.
    Currently, there is an internal compiler error on Cortex-M23 for the
    epilog processing of sign extension.

    This patch addresses the following CVE-2024-0151 for Armv8-M.baseline.

    gcc/ChangeLog:

            PR target/115253
            * config/arm/arm.cc (cmse_nonsecure_call_inline_register_clear):
            Sign extend for Thumb1.
            (thumb1_expand_prologue): Add zero/sign extend.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #6 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Torbjorn Svensson <azoff@gcc.gnu.org>:

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

commit r15-1201-gcf5f9171bae1f5f3034dc9a055b77446962f1a8c
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #6 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Torbjorn Svensson <azoff@gcc.gnu.org>:

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

commit r15-1201-gcf5f9171bae1f5f3034dc9a055b77446962f1a8c
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #7 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-14 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

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

commit r14-10306-ga657148995e1c46178637c8de82d1fab5474a37d
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Thu Jun 6 17:12:11 2024 +0200

    arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

    Properly handle zero and sign extension for Armv8-M.baseline as
    Cortex-M23 can have the security extension active.
    Currently, there is an internal compiler error on Cortex-M23 for the
    epilog processing of sign extension.

    This patch addresses the following CVE-2024-0151 for Armv8-M.baseline.

    gcc/ChangeLog:

            PR target/115253
            * config/arm/arm.cc (cmse_nonsecure_call_inline_register_clear):
            Sign extend for Thumb1.
            (thumb1_expand_prologue): Add zero/sign extend.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit 65bd0655ece268895e5018e393bafb769e201c78)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #8 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-14 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

https://gcc.gnu.org/g:9100e78ba28b1b69d1362d18088e897ca0f99594

commit r14-10307-g9100e78ba28b1b69d1362d18088e897ca0f99594
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit cf5f9171bae1f5f3034dc9a055b77446962f1a8c)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #9 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-13 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

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

commit r13-8844-gbf3ffb44355ca8aeea18c95a2b027023b3dab569
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Thu Jun 6 17:12:11 2024 +0200

    arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

    Properly handle zero and sign extension for Armv8-M.baseline as
    Cortex-M23 can have the security extension active.
    Currently, there is an internal compiler error on Cortex-M23 for the
    epilog processing of sign extension.

    This patch addresses the following CVE-2024-0151 for Armv8-M.baseline.

    gcc/ChangeLog:

            PR target/115253
            * config/arm/arm.cc (cmse_nonsecure_call_inline_register_clear):
            Sign extend for Thumb1.
            (thumb1_expand_prologue): Add zero/sign extend.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit 65bd0655ece268895e5018e393bafb769e201c78)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #10 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-13 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

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

commit r13-8845-gdfab6851eb557a47a5e61d00ad4c519072a69f61
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit cf5f9171bae1f5f3034dc9a055b77446962f1a8c)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #11 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

https://gcc.gnu.org/g:55c1687d542e40f0d4ad1d3dc624695a1854d967

commit r12-10550-g55c1687d542e40f0d4ad1d3dc624695a1854d967
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Thu Jun 6 17:12:11 2024 +0200

    arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

    Properly handle zero and sign extension for Armv8-M.baseline as
    Cortex-M23 can have the security extension active.
    Currently, there is an internal compiler error on Cortex-M23 for the
    epilog processing of sign extension.

    This patch addresses the following CVE-2024-0151 for Armv8-M.baseline.

    gcc/ChangeLog:

            PR target/115253
            * config/arm/arm.cc (cmse_nonsecure_call_inline_register_clear):
            Sign extend for Thumb1.
            (thumb1_expand_prologue): Add zero/sign extend.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit 65bd0655ece268895e5018e393bafb769e201c78)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #12 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

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

commit r12-10551-g3d9e4eedb6b1f43e5d0cd46c9aa06caf7c2d3500
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit cf5f9171bae1f5f3034dc9a055b77446962f1a8c)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #13 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

https://gcc.gnu.org/g:319081d614dec354ae415472121e0e8ebc4b1402

commit r11-11477-g319081d614dec354ae415472121e0e8ebc4b1402
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Thu Jun 6 17:12:11 2024 +0200

    arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

    Properly handle zero and sign extension for Armv8-M.baseline as
    Cortex-M23 can have the security extension active.
    Currently, there is an internal compiler error on Cortex-M23 for the
    epilog processing of sign extension.

    This patch addresses the following CVE-2024-0151 for Armv8-M.baseline.

    gcc/ChangeLog:

            PR target/115253
            * config/arm/arm.c (cmse_nonsecure_call_inline_register_clear):
            Sign extend for Thumb1.
            (thumb1_expand_prologue): Add zero/sign extend.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit 65bd0655ece268895e5018e393bafb769e201c78)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[cvs-commit at gcc dot gnu.org]

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

--- Comment #14 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Torbjorn Svensson
<azoff@gcc.gnu.org>:

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

commit r11-11478-gbf9c877c4c9939274520a3f694037a9921ba9878
Author: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
Date:   Fri Jun 7 10:42:22 2024 +0200

    testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

    For Armv8.1-M, the clearing of the registers is handled differently than
    for Armv8-M, so update the test case accordingly.

    gcc/testsuite/ChangeLog:

            PR target/115253
            * gcc.target/arm/cmse/extend-return.c: Update test case
            condition for Armv8.1-M.

    Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
    Co-authored-by: Yvan ROUX <yvan.roux@foss.st.com>
    (cherry picked from commit cf5f9171bae1f5f3034dc9a055b77446962f1a8c)

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[azoff at gcc dot gnu.org]

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

Torbjorn SVENSSON <azoff at gcc dot gnu.org> changed:

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

--- Comment #15 from Torbjorn SVENSSON <azoff at gcc dot gnu.org> ---
Both the Cortex-M23 and the Cortex-M55 issues was resolved with the commits
listed in comment 5 though comment 14 (including the backporting to GCC11,
GCC12, GCC13 and GCC14 branches).

[Bug target/115253] [14/15 regression] New tests added by r14-10122-gad45086178d833 fail on Cortex M23 and M55 bare metal targets

[thiago.bauermann at linaro dot org]

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

--- Comment #16 from Thiago Jung Bauermann <thiago.bauermann at linaro dot org> ---
Thank you!