[Bug middle-end/47530] [trans-mem] tail call optimization problem with _ITM_commitTransaction

2012-06-04 Thread aldyh at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47530

--- Comment #10 from Aldy Hernandez aldyh at gcc dot gnu.org 2012-06-04 
15:51:22 UTC ---
Created attachment 27556
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27556
propsed patch being tested

Whoops, sorry Patrick.  I already have a patch I am testing :).  If you already
have yours tested, feel free to submit it, otherwise I'll post shortly.


[Bug middle-end/47530] [trans-mem] tail call optimization problem with _ITM_commitTransaction

2012-06-04 Thread patrick.marlier at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47530

--- Comment #11 from Patrick Marlier patrick.marlier at gmail dot com 
2012-06-04 15:58:35 UTC ---
Created attachment 27557
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27557
testcase

(In reply to comment #10)
 Created attachment 27556 [details]
 propsed patch being tested
 
 Whoops, sorry Patrick.  I already have a patch I am testing :).  If you 
 already
 have yours tested, feel free to submit it, otherwise I'll post shortly.

Go ahead and submit (I will not able to do it before few hours). My patch was
quite similar except for the testcase (using the tree dump but untested).
Testcase attached if you want to include it into the testsuite.


[Bug middle-end/47530] [trans-mem] tail call optimization problem with _ITM_commitTransaction

2012-06-04 Thread aldyh at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47530

--- Comment #12 from Aldy Hernandez aldyh at gcc dot gnu.org 2012-06-04 
16:51:33 UTC ---
Author: aldyh
Date: Mon Jun  4 16:51:24 2012
New Revision: 188190

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=188190
Log:
PR middle-end/47530
* trans-mem.c (expand_block_edges): Do not skip the first
statement when resetting the BB.


Added:
trunk/gcc/testsuite/g++.dg/tm/pr47530-2.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/trans-mem.c


[Bug middle-end/47530] [trans-mem] tail call optimization problem with _ITM_commitTransaction

2012-06-04 Thread aldyh at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47530

--- Comment #13 from Aldy Hernandez aldyh at gcc dot gnu.org 2012-06-04 
16:52:55 UTC ---
Author: aldyh
Date: Mon Jun  4 16:52:47 2012
New Revision: 188191

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=188191
Log:
PR middle-end/47530
* trans-mem.c (expand_block_edges): Do not skip the first
statement when resetting the BB.


Added:
branches/gcc-4_7-branch/gcc/testsuite/g++.dg/tm/pr47530-2.C
Modified:
branches/gcc-4_7-branch/gcc/ChangeLog
branches/gcc-4_7-branch/gcc/trans-mem.c


[Bug middle-end/47530] [trans-mem] tail call optimization problem with _ITM_commitTransaction

2012-06-04 Thread aldyh at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47530

Aldy Hernandez aldyh at gcc dot gnu.org changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

--- Comment #14 from Aldy Hernandez aldyh at gcc dot gnu.org 2012-06-04 
16:56:21 UTC ---
fixed on 4.7 and trunk.  closing PR.