Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-09 Thread Patrick Marlier
On 11/09/2011 03:23 PM, Jakub Jelinek wrote: On Wed, Nov 09, 2011 at 09:32:28AM -0500, Patrick Marlier wrote: * gcc.dg/tm/memopt-1.c: Adjust regexp. This results in ERROR: (DejaGnu) proc "248" does not exist. - [] is tcl procedure invocation. Testing following, will commit soon if

Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-09 Thread Jakub Jelinek
On Wed, Nov 09, 2011 at 09:32:28AM -0500, Patrick Marlier wrote: > > * gcc.dg/tm/memopt-1.c: Adjust regexp. This results in ERROR: (DejaGnu) proc "248" does not exist. - [] is tcl procedure invocation. Testing following, will commit soon if it succeeds: 2011-11-09 Jakub Jelinek

Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-09 Thread Patrick Marlier
On 11/08/2011 05:25 PM, Richard Henderson wrote: On 11/08/2011 02:08 PM, Patrick Marlier wrote: - change the match for g to _ITM_RU[48] Change the match to [248]. I have never seen a "long" type to have a size of 2 bytes but I am probably wrong. (I did not find the C specification but I fou

Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-08 Thread Richard Henderson
On 11/08/2011 02:08 PM, Patrick Marlier wrote: > - change the match for g to _ITM_RU[48] Change the match to [248]. r~

Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-08 Thread Patrick Marlier
On 11/08/2011 04:29 PM, Richard Henderson wrote: On 11/08/2011 12:21 PM, Patrick Marlier wrote: The other way could be to change the regex but we don't want that. Why not? My guess was that the purpose to have "long" in the testcase was to test the generation of another TM-builtin than _ITM

Re: [trans-mem] fix memopt-1.c for 32bits

2011-11-08 Thread Richard Henderson
On 11/08/2011 12:21 PM, Patrick Marlier wrote: > The other way could be to change the regex but we don't want that. Why not? r~