Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-26 Thread James Greenhalgh
On Wed, Jun 24, 2015 at 09:28:34PM +0100, Mikhail Maltsev wrote: On 06/24/2015 06:52 PM, Christophe Lyon wrote: Hi Mikhail, In the gcc-5-branch, I can see that your new inhibit-warn-2.C test fails (targets ARM and AArch64). I can see this error message in g++.log:

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-26 Thread Jason Merrill
On 06/24/2015 04:28 PM, Mikhail Maltsev wrote: It can probably be fixed like this: diff --git a/gcc/testsuite/g++.dg/diagnostic/inhibit-warn-2.C b/gcc/testsuite/g++.dg/diagnostic/inhibit-warn-2.C index cb16b4c..f658c1d 100644 --- a/gcc/testsuite/g++.dg/diagnostic/inhibit-warn-2.C +++

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-24 Thread Mikhail Maltsev
On 06/24/2015 06:52 PM, Christophe Lyon wrote: Hi Mikhail, In the gcc-5-branch, I can see that your new inhibit-warn-2.C test fails (targets ARM and AArch64). I can see this error message in g++.log: /aci-gcc-fsf/sources/gcc-fsf/gccsrc/gcc/testsuite/g++.dg/diagnostic/inhibit-warn-2.C: In

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-24 Thread Christophe Lyon
On 22 June 2015 at 18:59, Jason Merrill ja...@redhat.com wrote: On 06/19/2015 08:23 PM, Mikhail Maltsev wrote: I see that version 5.2 is set as target milestone for this bug. Should I backport the patch? Please. Jason Hi Mikhail, In the gcc-5-branch, I can see that your new

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-22 Thread Jason Merrill
On 06/19/2015 08:23 PM, Mikhail Maltsev wrote: I see that version 5.2 is set as target milestone for this bug. Should I backport the patch? Please. Jason

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-19 Thread Mikhail Maltsev
On 19.06.2015 19:35, Jason Merrill wrote: OK, thanks. Sorry this took so long to review; please feel free to ping me every week. Jason I added the testcase from PR66467, bootstrapped and regtested on x86_64-linux. The final variant is attached. I applied it to trunk. I see that version

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-19 Thread Jason Merrill
OK, thanks. Sorry this took so long to review; please feel free to ping me every week. Jason

Re: [Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-06-09 Thread Marek Polacek
This is not my patch, but I'd like to ping it anyway as it also fixes PR66467 (you might want to add a testcase from this PR as well). On Sun, Apr 26, 2015 at 02:11:14PM +0300, Mikhail Maltsev wrote: The attached patch was bootstrapped and regtested on x86_64-unknown-linux-gnu. -- Regards,

[Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-04-26 Thread Mikhail Maltsev
The attached patch was bootstrapped and regtested on x86_64-unknown-linux-gnu. -- Regards, Mikhail Maltsev gcc/cp/ChangeLog: 2015-04-26 Mikhail Maltsev malts...@gmail.com * call.c (build_new_op_1): Check tf_warning flag in all cases gcc/testsuite/ChangeLog: 2015-04-26 Mikhail