Re: [PATCH, C++, PR58282] Handle noexcept on transactions with -fno-exceptions

2013-10-25 Thread Tom de Vries
On 07/09/13 18:54, Jason Merrill wrote: OK. I've reproduced the same problem with the 4.7 and 4.8 branch, and checked that applying the patch fixes the problem. Committed to 4.7 and 4.8 branch as well. Thanks, - Tom

Re: [PATCH, C++, PR58282] Handle noexcept on transactions with -fno-exceptions

2013-09-07 Thread Jason Merrill
OK. Jason

Re: [PATCH, C++, PR58282] Handle noexcept on transactions with -fno-exceptions

2013-09-06 Thread Tom de Vries
On 04/09/13 19:21, Jason Merrill wrote: On 09/03/2013 06:03 AM, Tom de Vries wrote: * semantics.c (finish_transaction_stmt, build_transaction_expr): Handle flag_exceptions. I'd rather handle this at a lower level, by making build_must_not_throw_expr return its argument if

Re: [PATCH, C++, PR58282] Handle noexcept on transactions with -fno-exceptions

2013-09-04 Thread Jason Merrill
On 09/03/2013 06:03 AM, Tom de Vries wrote: * semantics.c (finish_transaction_stmt, build_transaction_expr): Handle flag_exceptions. I'd rather handle this at a lower level, by making build_must_not_throw_expr return its argument if -fno-exceptions. Jason

[PATCH, C++, PR58282] Handle noexcept on transactions with -fno-exceptions

2013-09-03 Thread Tom de Vries
Jason, Consider this testcase reduced from g++.dg/tm/noexcept-1.C: ... struct TrueFalse { static constexpr bool v() { return true; } }; int global; templatetypename T int foo() { __transaction_atomic noexcept(T::v()) { global += 1; } return __transaction_atomic noexcept(T::v()) (global +