* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
@ 2014-11-21 13:56 ` jakub at gcc dot gnu.org
2014-11-21 20:42 ` jakub at gcc dot gnu.org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-11-21 13:56 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
2014-11-21 13:56 ` [Bug target/63975] " jakub at gcc dot gnu.org
@ 2014-11-21 20:42 ` jakub at gcc dot gnu.org
2014-11-21 20:43 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-11-21 20:42 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
2014-11-21 13:56 ` [Bug target/63975] " jakub at gcc dot gnu.org
2014-11-21 20:42 ` jakub at gcc dot gnu.org
@ 2014-11-21 20:43 ` jakub at gcc dot gnu.org
2014-11-23 4:12 ` pinskia at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-11-21 20:43 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
` (2 preceding siblings ...)
2014-11-21 20:43 ` jakub at gcc dot gnu.org
@ 2014-11-23 4:12 ` pinskia at gcc dot gnu.org
2014-12-11 16:47 ` belagod at gcc dot gnu.org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: pinskia at gcc dot gnu.org @ 2014-11-23 4:12 UTC (permalink / raw)
To: gcc-bugs
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 ***
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
` (3 preceding siblings ...)
2014-11-23 4:12 ` pinskia at gcc dot gnu.org
@ 2014-12-11 16:47 ` belagod at gcc dot gnu.org
2014-12-11 22:33 ` pinskia at gcc dot gnu.org
2015-10-29 23:19 ` mshawcroft at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: belagod at gcc dot gnu.org @ 2014-12-11 16:47 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
` (4 preceding siblings ...)
2014-12-11 16:47 ` belagod at gcc dot gnu.org
@ 2014-12-11 22:33 ` pinskia at gcc dot gnu.org
2015-10-29 23:19 ` mshawcroft at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: pinskia at gcc dot gnu.org @ 2014-12-11 22:33 UTC (permalink / raw)
To: gcc-bugs
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
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug target/63975] some of the builtin-arith-overflow* fail on aarch64
2014-11-19 18:23 [Bug target/63975] New: some of the builtin-arith-overflow* fail on aarch64 pinskia at gcc dot gnu.org
` (5 preceding siblings ...)
2014-12-11 22:33 ` pinskia at gcc dot gnu.org
@ 2015-10-29 23:19 ` mshawcroft at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: mshawcroft at gcc dot gnu.org @ 2015-10-29 23:19 UTC (permalink / raw)
To: gcc-bugs
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.
^ permalink raw reply [flat|nested] 8+ messages in thread