* [Bug ada/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
@ 2014-11-24 15:08 ` ro at gcc dot gnu.org
2014-11-24 16:07 ` rguenth at gcc dot gnu.org
` (8 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ro at gcc dot gnu.org @ 2014-11-24 15:08 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
Rainer Orth <ro at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |5.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug ada/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
2014-11-24 15:08 ` [Bug ada/64055] " ro at gcc dot gnu.org
@ 2014-11-24 16:07 ` rguenth at gcc dot gnu.org
2014-11-24 23:06 ` dominiq at lps dot ens.fr
` (7 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-11-24 16:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P1
Status|UNCONFIRMED |NEW
Last reconfirmed| |2014-11-24
Ever confirmed|0 |1
--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Confirmed - same on x86_64-linux.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug ada/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
2014-11-24 15:08 ` [Bug ada/64055] " ro at gcc dot gnu.org
2014-11-24 16:07 ` rguenth at gcc dot gnu.org
@ 2014-11-24 23:06 ` dominiq at lps dot ens.fr
2014-11-25 12:12 ` ro at CeBiTec dot Uni-Bielefeld.DE
` (6 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: dominiq at lps dot ens.fr @ 2014-11-24 23:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #2 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
Revision r217101 is OK.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug ada/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (2 preceding siblings ...)
2014-11-24 23:06 ` dominiq at lps dot ens.fr
@ 2014-11-25 12:12 ` ro at CeBiTec dot Uni-Bielefeld.DE
2014-11-28 11:07 ` ebotcazou at gcc dot gnu.org
` (5 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2014-11-25 12:12 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
> --- Comment #2 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
> Revision r217101 is OK.
Not in my case: at r217915, the test FAILs for the 32-bit multilib of
both the i386-apple-darwin14.0.0 and x86_64-apple-darwin14.0.0
configurations. Same for i686-unknown-linux-gnu and
x86_64-unknown-linux-gnu and i386-pc-solaris2.1[01] and
amd64-pc-solaris2.1[01].
Also (only checked i386-pc-solaris2.11) at r216981 and r217189.
Rainer
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug ada/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (3 preceding siblings ...)
2014-11-25 12:12 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2014-11-28 11:07 ` ebotcazou at gcc dot gnu.org
2014-11-28 11:07 ` [Bug target/64055] " ebotcazou at gcc dot gnu.org
` (4 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2014-11-28 11:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #4 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
ix86_function_arg_advance calls chkp_type_bounds_count which dies on an array
type with variable bounds so the immediate workaround is:
Index: tree-chkp.c
===================================================================
--- tree-chkp.c (revision 218133)
+++ tree-chkp.c (working copy)
@@ -1565,7 +1565,9 @@ chkp_find_bound_slots_1 (const_tree type
HOST_WIDE_INT esize = TREE_INT_CST_LOW (TYPE_SIZE (etype));
unsigned HOST_WIDE_INT cur;
- if (!maxval || integer_minus_onep (maxval))
+ if (!maxval
+ || integer_minus_onep (maxval)
+ || CONTAINS_PLACEHOLDER_P (maxval))
return;
for (cur = 0; cur <= TREE_INT_CST_LOW (maxval); cur++)
but a specialist needs to have a look here.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug target/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (4 preceding siblings ...)
2014-11-28 11:07 ` ebotcazou at gcc dot gnu.org
@ 2014-11-28 11:07 ` ebotcazou at gcc dot gnu.org
2014-11-28 15:40 ` enkovich.gnu at gmail dot com
` (3 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2014-11-28 11:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ebotcazou at gcc dot gnu.org
Component|ada |target
--- Comment #5 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
Recategorizing.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug target/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (5 preceding siblings ...)
2014-11-28 11:07 ` [Bug target/64055] " ebotcazou at gcc dot gnu.org
@ 2014-11-28 15:40 ` enkovich.gnu at gmail dot com
2014-11-28 17:02 ` ebotcazou at gcc dot gnu.org
` (2 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-28 15:40 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #6 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
TREE_INT_CST_LOW (maxval) assumes integer constant anyway. Therefore we may
use simpler check. It fixes gnat.dg/derived_aggregate.adb.
diff --git a/gcc/tree-chkp.c b/gcc/tree-chkp.c
index 0fb78cc..84886da 100644
--- a/gcc/tree-chkp.c
+++ b/gcc/tree-chkp.c
@@ -1568,7 +1568,9 @@ chkp_find_bound_slots_1 (const_tree type, bitmap
have_bound,
HOST_WIDE_INT esize = TREE_INT_CST_LOW (TYPE_SIZE (etype));
unsigned HOST_WIDE_INT cur;
- if (!maxval || integer_minus_onep (maxval))
+ if (!maxval
+ || TREE_CODE (maxval) != INTEGER_CST
+ || integer_minus_onep (maxval))
return;
for (cur = 0; cur <= TREE_INT_CST_LOW (maxval); cur++)
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug target/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (6 preceding siblings ...)
2014-11-28 15:40 ` enkovich.gnu at gmail dot com
@ 2014-11-28 17:02 ` ebotcazou at gcc dot gnu.org
2014-12-01 12:43 ` ienkovich at gcc dot gnu.org
2015-01-12 10:18 ` rguenth at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2014-11-28 17:02 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #7 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
> TREE_INT_CST_LOW (maxval) assumes integer constant anyway. Therefore we may
> use simpler check. It fixes gnat.dg/derived_aggregate.adb.
FWIW fine with me.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug target/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (7 preceding siblings ...)
2014-11-28 17:02 ` ebotcazou at gcc dot gnu.org
@ 2014-12-01 12:43 ` ienkovich at gcc dot gnu.org
2015-01-12 10:18 ` rguenth at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2014-12-01 12:43 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
--- Comment #8 from ienkovich at gcc dot gnu.org ---
Author: ienkovich
Date: Mon Dec 1 12:43:04 2014
New Revision: 218207
URL: https://gcc.gnu.org/viewcvs?rev=218207&root=gcc&view=rev
Log:
PR target/64055
* tree-chkp.c (chkp_find_bound_slots_1): Allow non constant
values in array domain.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-chkp.c
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug target/64055] [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386
2014-11-24 15:08 [Bug ada/64055] New: [5 regression] gnat.dg/derived_aggregate.adb FAILs on 32-bit i386 ro at gcc dot gnu.org
` (8 preceding siblings ...)
2014-12-01 12:43 ` ienkovich at gcc dot gnu.org
@ 2015-01-12 10:18 ` rguenth at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: rguenth at gcc dot gnu.org @ 2015-01-12 10:18 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64055
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #9 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.
^ permalink raw reply [flat|nested] 11+ messages in thread