On 10/19/2011 03:16 AM, Torvald Riegel wrote:
> On Tue, 2011-10-18 at 14:29 -0500, Aldy Hernandez wrote:
>>> Several ICEs in the TM tests on C++, but I think they are old. C TM
>>> tests work except some missing optimizations (old failures too).
>>> c-c++-common/tm/wrap-[12].c fail on C++, but not on C:
>>> wrap-1.c:5:57: error: 'transaction_wrap' argument not an identifier
>>> I believe this is an old error too?
>>> (I don't yet have a testcase summary of before the changes, so I can't
>>> compare properly. But will send an follow-up if there are regressions.)
>>
>> Can you run the tests before your patch to make sure there are no 
>> regressions?  That is, let's be sure that these are indeed old failures.
> 
> Indeed old failures, no new regressions. Tested and boot-strapped on
> x86_64.
> 
> OK for branch?
> 

Yep.


r~

Reply via email to