public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements
@ 2011-12-13 15:26 jamborm at gcc dot gnu.org
2011-12-13 15:32 ` [Bug tree-optimization/51528] " jamborm at gcc dot gnu.org
` (17 more replies)
0 siblings, 18 replies; 19+ messages in thread
From: jamborm at gcc dot gnu.org @ 2011-12-13 15:26 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Bug #: 51528
Summary: SRA should not create BOOLEAN_TYPE replacements
Classification: Unclassified
Product: gcc
Version: 4.7.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: tree-optimization
AssignedTo: jamborm@gcc.gnu.org
ReportedBy: jamborm@gcc.gnu.org
We already do not create ENUMERAL_TYPE replacements in SRA and neither
we should be creating BOOLEAN_TYPE ones because that can lead to
miscompilations, as the attached testcase shows. Like in the enumeral
case, we should only create replacements of types with precision that
matches their mode size.
I'll take care of that.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
@ 2011-12-13 15:32 ` jamborm at gcc dot gnu.org
2011-12-13 15:39 ` jamborm at gcc dot gnu.org
` (16 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: jamborm at gcc dot gnu.org @ 2011-12-13 15:32 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
--- Comment #1 from Martin Jambor <jamborm at gcc dot gnu.org> 2011-12-13 15:26:14 UTC ---
Created attachment 26071
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26071
Testcase
This is the aforementioned test case.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
2011-12-13 15:32 ` [Bug tree-optimization/51528] " jamborm at gcc dot gnu.org
@ 2011-12-13 15:39 ` jamborm at gcc dot gnu.org
2012-01-04 0:41 ` [Bug tree-optimization/51528] [4.6/4.7 Regression] " pinskia at gcc dot gnu.org
` (15 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: jamborm at gcc dot gnu.org @ 2011-12-13 15:39 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Martin Jambor <jamborm at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2011-12-13
Ever Confirmed|0 |1
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
2011-12-13 15:32 ` [Bug tree-optimization/51528] " jamborm at gcc dot gnu.org
2011-12-13 15:39 ` jamborm at gcc dot gnu.org
@ 2012-01-04 0:41 ` pinskia at gcc dot gnu.org
2012-01-04 10:18 ` rguenth at gcc dot gnu.org
` (14 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: pinskia at gcc dot gnu.org @ 2012-01-04 0:41 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Andrew Pinski <pinskia at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |wrong-code
Summary|SRA should not create |[4.6/4.7 Regression] SRA
|BOOLEAN_TYPE replacements |should not create
| |BOOLEAN_TYPE replacements
--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-01-04 00:40:53 UTC ---
I saw this on mips64-linux-gnu with one of the fortran testcases.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (2 preceding siblings ...)
2012-01-04 0:41 ` [Bug tree-optimization/51528] [4.6/4.7 Regression] " pinskia at gcc dot gnu.org
@ 2012-01-04 10:18 ` rguenth at gcc dot gnu.org
2012-01-04 13:42 ` rguenth at gcc dot gnu.org
` (13 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-04 10:18 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.6.3
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (3 preceding siblings ...)
2012-01-04 10:18 ` rguenth at gcc dot gnu.org
@ 2012-01-04 13:42 ` rguenth at gcc dot gnu.org
2012-01-27 13:09 ` rguenth at gcc dot gnu.org
` (12 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-04 13:42 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P2
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (4 preceding siblings ...)
2012-01-04 13:42 ` rguenth at gcc dot gnu.org
@ 2012-01-27 13:09 ` rguenth at gcc dot gnu.org
2012-01-30 13:41 ` rguenth at gcc dot gnu.org
` (11 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-27 13:09 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|jamborm at gcc dot gnu.org |rguenth at gcc dot gnu.org
--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-01-27 13:05:40 UTC ---
SRA creates, for
struct S { int i; int j; };
struct S bar (struct S);
struct S foo1 (int i)
{
struct S x;
x.i = i;
x = bar (x);
/* LHS disqualified, RHS partially scalarized. */
return x;
}
for the aggregate copy to the return slot
x$i_8 = x.i;
D.1720 = x;
D.1720.i = x$i_8;
but that's needlessly complicated as it should just have created
x.i = x$i_8;
D.1720 = x;
or for the similar case
struct S g;
struct S foo3 (int i)
{
struct S x;
/* RHS disqualified, LHS partially scalarized. */
x = g;
x.i = i;
x = bar (x);
return x;
}
it creates
x = g;
x$i_4 = g.i;
x$i_8 = i_1(D);
x.i = x$i_8;
which is just a complicated from of
x = g;
x$i_4 = x.i;
a sub-case which it handles fine when handling calls!
x = bar (x);
x$i_10 = x.i;
That copy-in-out game to the unscalarized parts causes this regression.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (5 preceding siblings ...)
2012-01-27 13:09 ` rguenth at gcc dot gnu.org
@ 2012-01-30 13:41 ` rguenth at gcc dot gnu.org
2012-01-31 10:08 ` rguenth at gcc dot gnu.org
` (10 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-30 13:41 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
--- Comment #4 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-01-30 13:26:48 UTC ---
Author: rguenth
Date: Mon Jan 30 13:26:45 2012
New Revision: 183720
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=183720
Log:
2012-01-30 Richard Guenther <rguenther@suse.de>
PR tree-optimization/51528
* tree-sra.c (sra_modify_assign): Re-factor in preparation
for PR51528 fix.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-sra.c
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6/4.7 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (6 preceding siblings ...)
2012-01-30 13:41 ` rguenth at gcc dot gnu.org
@ 2012-01-31 10:08 ` rguenth at gcc dot gnu.org
2012-01-31 10:54 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
` (9 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-31 10:08 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
--- Comment #5 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-01-31 09:46:36 UTC ---
Author: rguenth
Date: Tue Jan 31 09:46:29 2012
New Revision: 183752
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=183752
Log:
2012-01-31 Richard Guenther <rguenther@suse.de>
PR tree-optimization/51528
* tree-sra.c (sra_modify_assign): Avoid copy-in/out for aggregate
assigns.
* gcc.dg/torture/pr51528.c: New testcase.
Added:
trunk/gcc/testsuite/gcc.dg/torture/pr51528.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-sra.c
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (7 preceding siblings ...)
2012-01-31 10:08 ` rguenth at gcc dot gnu.org
@ 2012-01-31 10:54 ` rguenth at gcc dot gnu.org
2012-02-04 19:15 ` dominiq at lps dot ens.fr
` (8 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-01-31 10:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Known to work| |4.7.0
Summary|[4.6/4.7 Regression] SRA |[4.6 Regression] SRA should
|should not create |not create BOOLEAN_TYPE
|BOOLEAN_TYPE replacements |replacements
--- Comment #6 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-01-31 10:43:09 UTC ---
The testcase is worked around. Maybe reliably enough, maybe not ...
At least the patches seem to be backportable eventually.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (8 preceding siblings ...)
2012-01-31 10:54 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
@ 2012-02-04 19:15 ` dominiq at lps dot ens.fr
2012-02-06 14:14 ` rguenth at gcc dot gnu.org
` (7 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: dominiq at lps dot ens.fr @ 2012-02-04 19:15 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Dominique d'Humieres <dominiq at lps dot ens.fr> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rguenther at suse dot de
--- Comment #7 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-02-04 19:14:34 UTC ---
On powerpc-apple-darwin9 testing revision 183874 I get with -m32:
FAIL: gcc.dg/torture/pr51528.c -O2 execution test
FAIL: gcc.dg/torture/pr51528.c -O3 -fomit-frame-pointer execution test
FAIL: gcc.dg/torture/pr51528.c -O3 -g execution test
FAIL: gcc.dg/torture/pr51528.c -Os execution test
FAIL: gcc.dg/torture/pr51528.c -O2 -flto -flto-partition=none execution test
FAIL: gcc.dg/torture/pr51528.c -O2 -flto execution test
These execution failures are due to a.i==0.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (9 preceding siblings ...)
2012-02-04 19:15 ` dominiq at lps dot ens.fr
@ 2012-02-06 14:14 ` rguenth at gcc dot gnu.org
2012-02-13 17:47 ` iains at gcc dot gnu.org
` (6 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-02-06 14:14 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |52115
--- Comment #8 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-06 14:13:48 UTC ---
On ppc the main SRA pass does nothing to foo () because of differences in
how the return value is returned. Still the IL at expansion time looks
great:
foo ()
{
<bb 2>:
<retval>.b = 1;
use_bool (<retval>);
<retval>.i = 65534;
return <retval>;
}
compared to x86_64 where we have the uglier
foo ()
{
union U u;
union U D.1733;
<bb 2>:
u.b = 1;
use_bool (u);
u = VIEW_CONVERT_EXPR<union U>(65534);
D.1733 = u;
u ={v} {CLOBBER};
return D.1733;
Compared to -fno-tree-sra on PPC we have
foo ()
{
union U u;
<bb 2>:
<retval>.b = 1;
use_bool (<retval>);
u.i = 65534;
<retval> = u;
u ={v} {CLOBBER};
return <retval>;
so SRA managed to remove the aggregate temporary and its copy.
So, I don't see what's wrong on PPC (with SRA, that is). This must be
a target bug if at all (note I'm only inspecting dumps and did not
reproduce the execute fail).
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (10 preceding siblings ...)
2012-02-06 14:14 ` rguenth at gcc dot gnu.org
@ 2012-02-13 17:47 ` iains at gcc dot gnu.org
2012-02-13 20:42 ` iains at gcc dot gnu.org
` (5 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: iains at gcc dot gnu.org @ 2012-02-13 17:47 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Iain Sandoe <iains at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |iains at gcc dot gnu.org
--- Comment #9 from Iain Sandoe <iains at gcc dot gnu.org> 2012-02-13 17:46:42 UTC ---
the target has a different sized bool if that is relevant (int 32 sized)...
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (11 preceding siblings ...)
2012-02-13 17:47 ` iains at gcc dot gnu.org
@ 2012-02-13 20:42 ` iains at gcc dot gnu.org
2012-02-14 15:35 ` rguenth at gcc dot gnu.org
` (4 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: iains at gcc dot gnu.org @ 2012-02-13 20:42 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
--- Comment #10 from Iain Sandoe <iains at gcc dot gnu.org> 2012-02-13 20:40:43 UTC ---
(In reply to comment #8)
> So, I don't see what's wrong on PPC (with SRA, that is). This must be
> a target bug if at all (note I'm only inspecting dumps and did not
> reproduce the execute fail).
yes, seems to be .. the tree-ssa dump is unchanged between -O1 (OK) and -O2
(fails).
this seems to be a 4.5, 4.6 and 4.7 regression [4.4. is OK] - the call to bar
() is being dropped from foo ().
will move to a different PR. Thanks for the test-case ;)
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.5/4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (13 preceding siblings ...)
2012-02-14 15:35 ` rguenth at gcc dot gnu.org
@ 2012-02-14 15:35 ` rguenth at gcc dot gnu.org
2012-02-14 15:39 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
` (2 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-02-14 15:35 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.6.3 |4.5.4
Summary|[4.6 Regression] SRA should |[4.5/4.6 Regression] SRA
|not create BOOLEAN_TYPE |should not create
|replacements |BOOLEAN_TYPE replacements
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (12 preceding siblings ...)
2012-02-13 20:42 ` iains at gcc dot gnu.org
@ 2012-02-14 15:35 ` rguenth at gcc dot gnu.org
2012-02-14 15:35 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
` (3 subsequent siblings)
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-02-14 15:35 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Bug 51528 depends on bug 52244, which changed state.
Bug 52244 Summary: [4.5/4.6/4.7 Regression] wrong code for function returning union between int and _Bool at O > 2, with no-early-inlining
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52244
What |Old Value |New Value
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |DUPLICATE
--- Comment #11 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-14 15:33:03 UTC ---
*** Bug 52244 has been marked as a duplicate of this bug. ***
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (14 preceding siblings ...)
2012-02-14 15:35 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
@ 2012-02-14 15:39 ` rguenth at gcc dot gnu.org
2012-07-02 12:17 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
2013-04-12 16:18 ` [Bug tree-optimization/51528] [4.6 " jakub at gcc dot gnu.org
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-02-14 15:39 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
--- Comment #12 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-14 15:34:02 UTC ---
Author: rguenth
Date: Tue Feb 14 15:33:56 2012
New Revision: 184214
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184214
Log:
2012-02-14 Richard Guenther <rguenther@suse.de>
PR tree-optimization/52244
PR tree-optimization/51528
* tree-sra.c (analyze_access_subtree): Only create INTEGER_TYPE
replacements for integral types.
* gcc.dg/torture/pr52244.c: New testcase.
Added:
trunk/gcc/testsuite/gcc.dg/torture/pr52244.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-sra.c
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.5/4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (15 preceding siblings ...)
2012-02-14 15:39 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
@ 2012-07-02 12:17 ` rguenth at gcc dot gnu.org
2013-04-12 16:18 ` [Bug tree-optimization/51528] [4.6 " jakub at gcc dot gnu.org
17 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-07-02 12:17 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.5.4 |4.6.4
--- Comment #13 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-07-02 12:16:53 UTC ---
The 4.5 branch is being closed, adjusting target milestone.
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Bug tree-optimization/51528] [4.6 Regression] SRA should not create BOOLEAN_TYPE replacements
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
` (16 preceding siblings ...)
2012-07-02 12:17 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
@ 2013-04-12 16:18 ` jakub at gcc dot gnu.org
17 siblings, 0 replies; 19+ messages in thread
From: jakub at gcc dot gnu.org @ 2013-04-12 16:18 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51528
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
Target Milestone|4.6.4 |4.7.0
--- Comment #14 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-04-12 16:18:04 UTC ---
The 4.6 branch has been closed, fixed in GCC 4.7.0.
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2013-04-12 16:18 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-13 15:26 [Bug tree-optimization/51528] New: SRA should not create BOOLEAN_TYPE replacements jamborm at gcc dot gnu.org
2011-12-13 15:32 ` [Bug tree-optimization/51528] " jamborm at gcc dot gnu.org
2011-12-13 15:39 ` jamborm at gcc dot gnu.org
2012-01-04 0:41 ` [Bug tree-optimization/51528] [4.6/4.7 Regression] " pinskia at gcc dot gnu.org
2012-01-04 10:18 ` rguenth at gcc dot gnu.org
2012-01-04 13:42 ` rguenth at gcc dot gnu.org
2012-01-27 13:09 ` rguenth at gcc dot gnu.org
2012-01-30 13:41 ` rguenth at gcc dot gnu.org
2012-01-31 10:08 ` rguenth at gcc dot gnu.org
2012-01-31 10:54 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
2012-02-04 19:15 ` dominiq at lps dot ens.fr
2012-02-06 14:14 ` rguenth at gcc dot gnu.org
2012-02-13 17:47 ` iains at gcc dot gnu.org
2012-02-13 20:42 ` iains at gcc dot gnu.org
2012-02-14 15:35 ` rguenth at gcc dot gnu.org
2012-02-14 15:35 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
2012-02-14 15:39 ` [Bug tree-optimization/51528] [4.6 " rguenth at gcc dot gnu.org
2012-07-02 12:17 ` [Bug tree-optimization/51528] [4.5/4.6 " rguenth at gcc dot gnu.org
2013-04-12 16:18 ` [Bug tree-optimization/51528] [4.6 " jakub at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).