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~