Re: PR/54893: allow volatiles inside relaxed transactions

2012-10-16 Thread Richard Henderson
On 2012-10-12 20:42, Aldy Hernandez wrote: PR middle-end/54893 * trans-mem.c (diagnose_tm_1_op): Allow volatiles inside relaxed transactions. Ok. r~

Re: PR/54893: allow volatiles inside relaxed transactions

2012-10-12 Thread Aldy Hernandez
On 10/11/12 17:15, Richard Henderson wrote: On 10/11/2012 01:56 PM, Aldy Hernandez wrote: PR middle-end/54893 * trans-mem.c (diagnose_tm_1_op): Allow volatiles inside relaxed transactions. Ok. r~ Sorry for the noise, but Torvald pointed out that the transaction

PR/54893: allow volatiles inside relaxed transactions

2012-10-11 Thread Aldy Hernandez
Apparently we were a bit too aggressive on disallowing volatiles inside transactions. Torvald seems to agree that volatiles should be allowed inside relaxed transactions, since they will go into serial irrevocable mode anyhow. The following patch fixes the PR. I did not include a test that

Re: PR/54893: allow volatiles inside relaxed transactions

2012-10-11 Thread Richard Henderson
On 10/11/2012 01:56 PM, Aldy Hernandez wrote: PR middle-end/54893 * trans-mem.c (diagnose_tm_1_op): Allow volatiles inside relaxed transactions. Ok. r~