[Bug middle-end/51949] [4.7 Regression] expand_call: seg fault caused by IPA split

2012-01-23 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51949

Richard Guenther rguenth at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|unassigned at gcc dot   |rguenth at gcc dot gnu.org
   |gnu.org |

--- Comment #3 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-23 
10:35:26 UTC ---
Mine.


[Bug middle-end/51949] [4.7 Regression] expand_call: seg fault caused by IPA split

2012-01-23 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51949

--- Comment #4 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-23 
10:42:04 UTC ---
We do not honor cur_node-local.can_change_signature, and that does not
take into account return value removal.  I have a smallish workaround.


[Bug middle-end/51949] [4.7 Regression] expand_call: seg fault caused by IPA split

2012-01-23 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51949

Richard Guenther rguenth at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #6 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-23 
10:53:13 UTC ---
Fixed.


[Bug middle-end/51949] [4.7 Regression] expand_call: seg fault caused by IPA split

2012-01-23 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51949

--- Comment #5 from Richard Guenther rguenth at gcc dot gnu.org 2012-01-23 
10:53:01 UTC ---
Author: rguenth
Date: Mon Jan 23 10:52:57 2012
New Revision: 183424

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=183424
Log:
2012-01-23  Richard Guenther  rguent...@suse.de

PR tree-optimization/51949
* ipa-split.c (execute_split_functions): Do not split malloc
functions.

* gcc.dg/torture/pr51949.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.dg/torture/pr51949.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/ipa-split.c
trunk/gcc/testsuite/ChangeLog


[Bug middle-end/51949] [4.7 Regression] expand_call: seg fault caused by IPA split

2012-01-22 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51949

Andrew Pinski pinskia at gcc dot gnu.org changed:

   What|Removed |Added

Summary|[4.7 Regression]|[4.7 Regression]
   |expand_call: seg fault  |expand_call: seg fault
   ||caused by IPA split

--- Comment #2 from Andrew Pinski pinskia at gcc dot gnu.org 2012-01-22 
21:24:56 UTC ---
The problem is IPA split is splitting the function mem_alloc correctly, though
it should have removed the malloc attribute on the split function.
This was introduced by:
2012-01-11  Eric Botcazou  ebotca...@adacore.com

...
* ipa-split.c (split_function): Skip the return value for the split
part if it doesn't return.

Though it was always there really.