[Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64

[Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64

[pinskia at gcc dot gnu.org]

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

            Bug ID: 63975
           Summary: some of the builtin-arith-overflow* fail on aarch64
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: pinskia at gcc dot gnu.org
            Target: aarch64

FAIL: c-c++-common/torture/builtin-arith-overflow-17.c   -O0  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-17.c   -O2  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-17.c   -O2 -flto
-fno-use-linker-plugin -flto-partition=none  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-17.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-18.c   -O0  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-18.c   -O2  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-18.c   -O2 -flto
-fno-use-linker-plugin -flto-partition=none  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-18.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-6.c   -O0  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-6.c   -O2  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-6.c   -O2 -flto
-fno-use-linker-plugin -flto-partition=none  execution test
FAIL: c-c++-common/torture/builtin-arith-overflow-6.c   -O2 -flto
-fuse-linker-plugin -fno-fat-lto-objects  execution test

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

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

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
All these tests involve __int128 arithmetics, might be as well a backend bug in
handling those.

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[jakub at gcc dot gnu.org]

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Fri Nov 21 20:42:20 2014
New Revision: 217946

URL: https://gcc.gnu.org/viewcvs?rev=217946&root=gcc&view=rev
Log:
    PR target/63848
    PR target/63975
    * internal-fn.c (expand_arith_overflow_result_store,
    expand_addsub_overflow, expand_neg_overflow, expand_mul_overflow): Use
    do_compare_rtx_and_jump instead of emit_cmp_and_jump_insns everywhere,
    adjust arguments to those functions.  Use unsignedp = true for
    EQ, NE, GEU, LEU, LTU and GTU comparisons.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/internal-fn.c

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63975
Bug 63975 depends on bug 63848, which changed state.

Bug 63848 Summary: [5 Regression] FAIL: c-c++-common/torture/builtin-arith-overflow-17.c   -O0  execution test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63848

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

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

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

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Actually it was the same as bug 63848.  Closing as dup and confirmed fixed.

*** This bug has been marked as a duplicate of bug 63848 ***

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[belagod at gcc dot gnu.org]

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

Tejas Belagod <belagod at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
   Last reconfirmed|                            |2014-12-11
                 CC|                            |belagod at gcc dot gnu.org
         Resolution|DUPLICATE                   |---
     Ever confirmed|0                           |1

--- Comment #4 from Tejas Belagod <belagod at gcc dot gnu.org> ---
aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never -O2
-fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
/work/dev/arm/src/gcc/gcc/testsuite/c-c++-common/torture/builtin-arith-overflow-10.c
-fPIC -o ba.exe

aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never -O2
-fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
/work/dev/arm/src/gcc/gcc/testsuite/c-c++-common/torture/builtin-arith-overflow-11.c
-fPIC -o ba.exe

aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never -O2
-fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
/work/dev/arm/src/gcc/gcc/testsuite/gcc.dg/builtin-arith-overflow-1.c -o ba.exe

still fail for me on aarch64.

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[pinskia at gcc dot gnu.org]

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

--- Comment #5 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Tejas Belagod from comment #4)
> aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never
> -O2 -fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
> /work/dev/arm/src/gcc/gcc/testsuite/c-c++-common/torture/builtin-arith-
> overflow-10.c -fPIC -o ba.exe
> 
> aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never
> -O2 -fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
> /work/dev/arm/src/gcc/gcc/testsuite/c-c++-common/torture/builtin-arith-
> overflow-11.c -fPIC -o ba.exe
> 
> aarch64-none-elf-gcc -fno-diagnostics-show-caret -fdiagnostics-color=never
> -O2 -fdump-tree-optimized -g -specs=aem-ve.specs -lm -mcmodel=small
> /work/dev/arm/src/gcc/gcc/testsuite/gcc.dg/builtin-arith-overflow-1.c -o
> ba.exe

-1.c was a bug in the testcase and has been fixed:
https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01049.html

[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

[mshawcroft at gcc dot gnu.org]

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

mshawcroft at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
                 CC|                            |mshawcroft at gcc dot gnu.org
         Resolution|---                         |FIXED

--- Comment #6 from mshawcroft at gcc dot gnu.org ---
This is already fixed.