[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[cvs-commit at gcc dot gnu.org]

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

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Stam Markianos-Wright
<stammark@gcc.gnu.org>:

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

commit r11-5332-gbc771e6c3a2b1fe74328639a00531fc0c3923e43
Author: Stam Markianos-Wright <stam.markianos-wright@arm.com>
Date:   Wed Nov 25 12:50:06 2020 +0000

    arm: Add test that was missing from old commit [PR91816]

    A while back I submitted GCC10 commit:

     44f77a6dea2f312ee1743f3dde465c1b8453ee13

    for PR91816.

    Turns out I was an idiot and forgot to include the test in the actual git
commit.

    Tested that the test still passes on a cross arm-none-eabi and also in a
    Cortex A-15 bootstrap with no regressions.

    gcc/testsuite/ChangeLog:
            PR target/91816
            * gcc.target/arm/pr91816.c: New test.

[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[cvs-commit at gcc dot gnu.org]

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

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Stam Markianos-Wright
<stammark@gcc.gnu.org>:

https://gcc.gnu.org/g:8804b6ae3da91f3baa82da2fe28090025d5717d3

commit r10-9079-g8804b6ae3da91f3baa82da2fe28090025d5717d3
Author: Stam Markianos-Wright <stam.markianos-wright@arm.com>
Date:   Wed Nov 25 13:11:09 2020 +0000

        arm: Add test that was missing from old commit [PR91816]

        A while back I submitted GCC10 commit:

         44f77a6dea2f312ee1743f3dde465c1b8453ee13

        for PR91816.

        Turns out I was an idiot and forgot to include the test in the actual
git commit.

        Tested that the test still passes on a cross arm-none-eabi and also in
a
        Cortex A-15 bootstrap with no regressions.

    gcc/testsuite/ChangeLog:

    2020-11-25  Stam Markianos-Wright  <stam.markianos-wright@arm.com>

            PR target/91816
            * gcc.target/arm/pr91816.c: New test.

[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[cvs-commit at gcc dot gnu.org]

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

--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-8 branch has been updated by Stam Markianos-Wright
<stammark@gcc.gnu.org>:

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

commit r8-10655-g3a936a9ecddef58ad5b6ee11697c3f791b942708
Author: Stam Markianos-Wright <stam.markianos-wright@arm.com>
Date:   Mon Nov 30 10:47:54 2020 +0000

    Backport of the patch for PR target/91816

    This is a patch for an issue where the compiler was generating a
    conditional branch in Thumb2, which was too far for b{cond} to handle.

    This backport also includes the subsequent fixes to the test in this
    patch.

    gcc/ChangeLog

            PR target/91816
            * config/arm/arm-protos.h: New function arm_gen_far_branch
prototype.
            * config/arm/arm.c (arm_gen_far_branch): New function
            arm_gen_far_branch.
            * config/arm/arm.md: Update b<cond> for Thumb2 range checks.

    gcc/testsuite/ChangeLog

            PR target/91816
            * gcc.target/arm/pr91816.c: New test.

[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[cvs-commit at gcc dot gnu.org]

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

--- Comment #8 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-9 branch has been updated by Stam Markianos-Wright
<stammark@gcc.gnu.org>:

https://gcc.gnu.org/g:6b7ab0e565d79a7e6ae5dbbf17a5eb4eafe56dc8

commit r9-9081-g6b7ab0e565d79a7e6ae5dbbf17a5eb4eafe56dc8
Author: Stam Markianos-Wright <stam.markianos-wright@arm.com>
Date:   Mon Nov 30 11:05:30 2020 +0000

    Backport of the patch for PR target/91816

    This is a patch for an issue where the compiler was generating a
    conditional branch in Thumb2, which was too far for b{cond} to handle.

    This backport also includes the subsequent fixes to the test in this
    patch.

    gcc/ChangeLog

            * config/arm/arm-protos.h: New function arm_gen_far_branch
prototype.
            * config/arm/arm.c (arm_gen_far_branch): New function
            arm_gen_far_branch.
            * config/arm/arm.md: Update b<cond> for Thumb2 range checks.

    gcc/testsuite/ChangeLog

            * gcc.target/arm/pr91816.c: New test.

[Bug target/91816] [8/9 Regression] Arm generates out of range conditional branches in Thumb2

[stammark at gcc dot gnu.org]

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

Stam Markianos-Wright <stammark at gcc dot gnu.org> changed:

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

--- Comment #9 from Stam Markianos-Wright <stammark at gcc dot gnu.org> ---
Patch is now on all active branches, so moving to RESOLVED. Thanks to all for
their reviews!