Re: [C++ Patch] PR 56725

2013-04-02 Thread Uros Bizjak
Hello! >> .. oops, the patch I attached has a typo. This is the right one. > > This regressed > FAIL: obj-c++.dg/try-catch-13.mm -fgnu-runtime (test for errors, line 12) > Can you please adjust that testcase too? Tested patch is at [1]. [1] http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00013.htm

Re: [C++ Patch] PR 56725

2013-04-02 Thread Paolo Carlini
On 04/02/2013 12:03 PM, Jakub Jelinek wrote: On Thu, Mar 28, 2013 at 03:55:38PM +0100, Paolo Carlini wrote: .. oops, the patch I attached has a typo. This is the right one. This regressed FAIL: obj-c++.dg/try-catch-13.mm -fgnu-runtime (test for errors, line 12) Can you please adjust that testc

Re: [C++ Patch] PR 56725

2013-04-02 Thread Jakub Jelinek
On Thu, Mar 28, 2013 at 03:55:38PM +0100, Paolo Carlini wrote: > .. oops, the patch I attached has a typo. This is the right one. This regressed FAIL: obj-c++.dg/try-catch-13.mm -fgnu-runtime (test for errors, line 12) Can you please adjust that testcase too? Jakub

Re: [C++ Patch] PR 56725

2013-03-28 Thread Jason Merrill
Ok. Jason

Re: [C++ Patch] PR 56725

2013-03-28 Thread Paolo Carlini
.. oops, the patch I attached has a typo. This is the right one. Paolo. // Index: cp/call.c === --- cp/call.c (revision 197198) +++ cp/call.c (working copy) @@ -5836,11 +5836,11 @@ convert_like_real (conversi

[C++ Patch] PR 56725

2013-03-28 Thread Paolo Carlini
Hi, in this PR Tom noticed that we are producing weird extra initial spaces in some error messages and Manuel replied that in fact those messages should rather be inform. Per Manuel suggestion, I therefore adjusted call.c in that sense but then noticed that eg, across rv1n.C, we were producin