[Bug tree-optimization/98123] New: if-to-switch tests fail on arm

[Bug tree-optimization/98123] New: if-to-switch tests fail on arm

[clyon at gcc dot gnu.org]

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

            Bug ID: 98123
           Summary: if-to-switch tests fail on arm
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: clyon at gcc dot gnu.org
  Target Milestone: ---

The recently introduced if-to-switch-* tests fail on arm:

gcc.dg/tree-ssa/if-to-switch-2.c scan-tree-dump iftoswitch "Condition chain
with 3 BBs transformed into a switch statement."
gcc.dg/tree-ssa/if-to-switch-3.c scan-tree-dump iftoswitch "Condition chain
with 3 BBs transformed into a switch statement."
gcc.dg/tree-ssa/if-to-switch-4.c scan-tree-dump-not iftoswitch "Condition
chain"
gcc.dg/tree-ssa/if-to-switch-5.c scan-tree-dump iftoswitch "Condition chain
with 5 BBs transformed into a switch statement."
gcc.dg/tree-ssa/if-to-switch-6.c scan-tree-dump-not iftoswitch "Condition
chain"
gcc.dg/tree-ssa/if-to-switch-8.c scan-tree-dump-not iftoswitch "Condition
chain"

When GCC is configured with:
--target arm-none-linux-gnueabihf
--with-mode thumb
--with-cpu cortex-a5
--with-fpu vfpv3-d16-fp16

Configuring for cortex-a9 instead of cortex-a5 shows less failures, only:
gcc.dg/tree-ssa/if-to-switch-4.c scan-tree-dump-not iftoswitch "Condition
chain"
gcc.dg/tree-ssa/if-to-switch-6.c scan-tree-dump-not iftoswitch "Condition
chain"
gcc.dg/tree-ssa/if-to-switch-8.c scan-tree-dump-not iftoswitch "Condition
chain"

[Bug tree-optimization/98123] if-to-switch tests fail on arm

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1
                 CC|                            |marxin at gcc dot gnu.org
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org
   Last reconfirmed|                            |2020-12-03

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
All right, I'm gonna test it with a cross compiler.

[Bug tree-optimization/98123] if-to-switch tests fail on arm

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |WAITING

--- Comment #2 from Martin Liška <marxin at gcc dot gnu.org> ---
Should be fixed on master, can you please re-check it?

[Bug tree-optimization/98123] if-to-switch tests fail on arm

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

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

--- Comment #3 from Martin Liška <marxin at gcc dot gnu.org> ---
Should be really fixed, please reopen if not.

[Bug tree-optimization/98123] if-to-switch tests fail on arm

[clyon at gcc dot gnu.org]

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

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

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

--- Comment #4 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Sorry I didn't reply earlier because I didn't have the most recent trunk
results yet.

What revision should have fixed them?
With r11-5718 (g:4a3b9f48c3743c6737acdc93074d058c1603be2a) I still see:
FAIL: gcc.dg/tree-ssa/if-to-switch-4.c scan-tree-dump-not iftoswitch "Condition
chain"
FAIL: gcc.dg/tree-ssa/if-to-switch-6.c scan-tree-dump-not iftoswitch "Condition
chain"
FAIL: gcc.dg/tree-ssa/if-to-switch-8.c scan-tree-dump-not iftoswitch "Condition
chain"

[Bug testsuite/98123] if-to-switch tests fail on arm

[cvs-commit at gcc dot gnu.org]

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

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:

https://gcc.gnu.org/g:485b40a5270cf50a7304a9e5cb8aff96bfa3d901

commit r11-5741-g485b40a5270cf50a7304a9e5cb8aff96bfa3d901
Author: Martin Liska <mliska@suse.cz>
Date:   Fri Dec 4 10:43:51 2020 +0100

    testsuite: use param for if-to-switch tests

    gcc/testsuite/ChangeLog:

            PR testsuite/98123
            * gcc.dg/tree-ssa/if-to-switch-4.c: Add param to make the test
            stable on all architectures.
            * gcc.dg/tree-ssa/if-to-switch-6.c: Likewise.
            * gcc.dg/tree-ssa/if-to-switch-8.c: Likewise.

[Bug testsuite/98123] if-to-switch tests fail on arm

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

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

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
Should be fixed now.