[PATCH PR60363]XFAIL case ssa-dom-thread-4.c for logical_op_short_circuit targets

2014-03-31 Thread bin.cheng
Hi, Patch for PR60363 is sent at http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00857.html , but it needs to wait for stage 1. In the meanwhile this patch is to xfail the case for logical_op_short_circuit targets. Is it ok? Thanks, bin gcc/testsuite/ChangeLog 2014-04-01 Bin Cheng PR t

[PING] [PATCH] Fix PR rtl-optimization/pr60663

2014-03-31 Thread Zhenqiang Chen
Ping? Bootstrap and no make check regression on X86-64. Bootstrap on ARM. In ARM regression test, some new PASS and FAIL of debug info check for gcc.dg/guality/pr36728-1.c and gcc.dg/guality/pr36728-2.c since register allocation result is different with the patch. There is no real new FAIL due to

Re: [C++ patch] for C++/52369

2014-03-31 Thread Jason Merrill
On 03/31/2014 04:27 PM, Fabien Chêne wrote: * cp/init.c (perform_member_init): Homogenize uninitialized diagnostics. OK. if (permerror (input_location, "default argument given for parameter " "%d of %q#D", i, newdecl)) perm

Re: [C++ patch] for C++/52369

2014-03-31 Thread Fabien Chêne
2014-03-28 17:59 GMT+01:00 Jason Merrill : [...] > The inform should only happen if permerror returns true (i.e. without > -fpermissive -w). OK with that change. I did a little tour around the permerror uses, and found that in decl.c: if (permerror (input_location, "defaul

Re: [C++ patch] for C++/52369

2014-03-31 Thread Fabien Chêne
2014-03-28 17:59 GMT+01:00 Jason Merrill : > On 03/27/2014 05:32 PM, Fabien Chêne wrote: >> >> + permerror (DECL_SOURCE_LOCATION (current_function_decl), >> +"uninitialized reference member in %q#T", type); >> + inform (DECL_SOURCE_LOCATION (member),

Re: [C++ patch] for C++/52369

2014-03-31 Thread Fabien Chêne
2014-03-28 20:27 GMT+01:00 Mike Stump : > Just a nit... > >> 2014-03-28 Fabien Chêne >> >>* cp/init.c (perform_member_init): homogeneize uninitialized >>diagnostics. > > Sentences begin with an upper case letter, and spelling... Homogenize.. I will fix that, thanks. -- Fabien

Re: Lost __mips_o32 predefine on NetBSD

2014-03-31 Thread Martin Husemann
On Mon, Mar 31, 2014 at 08:35:30PM +0100, Richard Sandiford wrote: > Are you sure it was ever in FSF GCC? I went through every revision > of netbsd.h in git and couldn't see it. No, I'm not - guess I never tried a pure gcc on mips. > I can go ahead and apply the patch anyway if it's the right th

Re: Lost __mips_o32 predefine on NetBSD

2014-03-31 Thread Richard Sandiford
Martin Husemann writes: > In the mips--netbsdelf target gcc 4.9 lost the pre-definition of > __mips_o32, which is heavily used in NetBSD sources. > > The obvious trivial patch adds it back. Are you sure it was ever in FSF GCC? I went through every revision of netbsd.h in git and couldn't see it.

Re: [patch, libgfortran] PR60128 Wrong ouput using en edit descriptor

2014-03-31 Thread Dominique d'Humières
Updated gfortran.dg/fmt_en.f90 to skip some tests not supported on i?86-*-solaris2.9* and hppa*-*-hpux* (these tests assume rounding to nearest and to even on tie, AFAICT i?86-*-solaris2.9* rounds real(16) to zero and hppa*-*-hpux* rounds all kinds to zero on tie, with some exceptions I don’t

Re: [PATCH] std::quoted is too eager to clear the string

2014-03-31 Thread Jonathan Wakely
On 21/02/14 01:00 +0100, Lars Gullik Bjønnes wrote: * include/std/iomanip (_Quoted_string operator>>): Do not clear string if input is not quoted. Currently the string is cleared regardless of how the string is handled. This patch just moves the clearing down so that we do not clear it if we

Re: [PATCH] Handle short writes / EINTR in simple_object_internal_write

2014-03-31 Thread Ian Lance Taylor
On Mon, Mar 31, 2014 at 3:14 AM, Richard Biener wrote: > > 2014-03-31 Richard Biener > > libiberty/ > * simple-object.c (simple_object_internal_write): Handle > EINTR and short writes. This is OK. Thanks. Ian

C++ PATCH for c++/60642 (abi_tag and templates)

2014-03-31 Thread Jason Merrill
This bug report pointed out that we weren't properly decorating template mangled names with the appropriate abi_tags. This patch fixes this by making sure that the mangler only looks at the abi_tag from the template, and the parser warns that we'll ignore tags on specializations and instantiat

Re: std::rethrow_exception is broken

2014-03-31 Thread Jonathan Wakely
On 31/03/14 19:17 +0100, Jonathan Wakely wrote: This improves the static assertion messages, as suggested by François and Daniel. Tested x86_64-linux, committed to trunk. Oops, apparently mutt doesn't attach attachments until you actually send the mail, by which time the file I was attaching h

Re: std::rethrow_exception is broken

2014-03-31 Thread Jonathan Wakely
On 29/03/14 19:55 +0100, Daniel Krügler wrote: 2014-03-28 13:37 GMT+01:00 Jonathan Wakely : On 27/03/14 23:00 +0100, Daniel Krügler wrote: I'm in favour for changing it, what about something like "layout mismatch" ? How about this? "__cxa_dependent_exception::termHandler layout must be cons

PR 60604: CANNOT_CHANGE_MODE_CLASS being ignored

2014-03-31 Thread Richard Sandiford
PR 60604 shows a case where CANNOT_CHANGE_MODE_CLASS is being ignored for a subreg of a floating-point register, causing it to be replaced with an invalid (reg ...). There are various reasons why MIPS floating-point registers can't change mode, but one important one for big-endian 32-bit targets i

Re: [PATCH] [4.8 branch] PR rtl-optimization/60700: Backport revision 201326

2014-03-31 Thread H.J. Lu
On Sat, Mar 29, 2014 at 7:34 AM, H.J. Lu wrote: > On Saturday, March 29, 2014, Jakub Jelinek wrote: >> >> On Sat, Mar 29, 2014 at 09:38:38AM +0100, Eric Botcazou wrote: >> > > Revision 201326 fixes a shrink-wrap bug which is also a regression >> > > on 4.8 branch. This patch backports it to 4.8

Re: patch to fix PR 60650

2014-03-31 Thread Ramana Radhakrishnan
On 03/29/14 11:24, Jakub Jelinek wrote: On Thu, Mar 27, 2014 at 02:51:38PM -0400, Vladimir Makarov wrote: 2014-03-27 Vladimir Makarov PR rtl-optimization/60650 * gcc.target/arm/pr60650.c: New. The test fails for me with: spawn -ignore SIGHUP /builddir/build/BUILD/gcc-4.9

Re: [PATCH] Guard special installs in install-driver

2014-03-31 Thread Joseph S. Myers
On Mon, 31 Mar 2014, Richard Biener wrote: > The following makes install-driver behave sanely when GCC_INSTALL_NAME > and GCC_TARGET_INSTALL_NAME match (or GCC_INSTALL_NAME and > $(target_noncanonical)-gcc-$(version)). We seem to force both > $(target_noncanonical)-gcc and $(target_noncanonical)-

Re: [PATCH v2] SPARC: Clarify -mapp-regs option

2014-03-31 Thread Sebastian Huber
On 2014-03-31 11:59, Richard Biener wrote: On Sun, Mar 16, 2014 at 12:43 PM, Sebastian Huber wrote: >gcc/ChangeLog >2014-03-16 Sebastian Huber > > * doc/invoke.texi (mapp-regs): Clarify. Ok. Thanks, Richard. I don't have commit access, so can someone please do this for me. -- Se

[PATCH][LTO] Fix PR60720

2014-03-31 Thread Richard Biener
The following patch makes sure we wrap symbol references in global initializers the same as we do for IL references - inside a MEM_REF that guards their context against type changes due to symbol merging (and thus in this case against ICEs when processing an ARRAY_REF around a non-array object).

Re: std::rethrow_exception is broken

2014-03-31 Thread Rainer Orth
Hi Jonathan, > On 31/03/14 14:13 +0200, Rainer Orth wrote: >> >>Unfortunately, the new tests FAIL on non-C99 targets (i386-pc-solaris2.9 >>in my case): >> >>FAIL: 18_support/exception_ptr/60612-terminate.cc (test for excess errors) >>Excess errors: >>/vol/gcc/src/hg/trunk/local/libstdc++-v3/testsu

[PATCH] Fix PR c++/44859

2014-03-31 Thread Patrick Palka
Hi, In PR c++/44859, it is noted that the following code does not emit a "returning reference to temporary" warning: struct Base2 { int m_foo; }; struct Derived2 : public Base2 {}; const Base2& f8() { return Derived2(); } // <-- here The source of the problem seems to be that maybe_warn_ab

[C++] Weffc++/Wnon-virtual-dtor confusion

2014-03-31 Thread Nathan Sidwell
This patch fixes some unexpected behaviour of the Weffc++ and Wnon-virtual-dtor flags. The documentation for the latter says it's also enabled by Weffc++, but that's untrue. The current behaviour of Weffc++ is to warn about any non-virtual dtor in a non-polymorphic base class (relying on the

Re: std::rethrow_exception is broken

2014-03-31 Thread Jonathan Wakely
On 31/03/14 14:13 +0200, Rainer Orth wrote: Unfortunately, the new tests FAIL on non-C99 targets (i386-pc-solaris2.9 in my case): FAIL: 18_support/exception_ptr/60612-terminate.cc (test for excess errors) Excess errors: /vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/18_support/exception_ptr

Re: std::rethrow_exception is broken

2014-03-31 Thread Rainer Orth
Jonathan Wakely writes: > On 25/03/14 17:25 +, Jonathan Wakely wrote: >>Tested x86_64-linux, I plan to commit this to trunk soon. >> > >>commit 06a845f80204947afd6866109db58cc85dc87117 >>Author: Jonathan Wakely >>Date: Tue Mar 25 14:42:45 2014 + >> >> PR libstdc++/60612 >> *

[PATCH] Guard special installs in install-driver

2014-03-31 Thread Richard Biener
The following makes install-driver behave sanely when GCC_INSTALL_NAME and GCC_TARGET_INSTALL_NAME match (or GCC_INSTALL_NAME and $(target_noncanonical)-gcc-$(version)). We seem to force both $(target_noncanonical)-gcc and $(target_noncanonical)-gcc-$(version) installs, but still apply $program_t

Re: various _mm512_set* intrinsics

2014-03-31 Thread Kirill Yukhin
Hello guys, On 28 Mar 20:10, Uros Bizjak wrote: > Hello! > > > Here are more intrinsics that are missing. I know that gcc currently > > generates horrible code for most of them but I think it's more important > > to have the API in place, albeit non-optimal. Maybe this entices some > > one to ad

[PATCH] Handle short writes / EINTR in simple_object_internal_write

2014-03-31 Thread Richard Biener
As promised here is the followup to handle the same kind of "errors" in simple_object_internal_write as in simple_object_internal_read. Bootstrapped on x86_64-unknown-linux-gnu, LTO bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk? Thanks, Richard. 2014-03-31 Richard Biener

[PATCH] Fix gcc.dg/lto/pr55113_0.c testsuite ICE

2014-03-31 Thread Richard Biener
This fixes gcc.dg/lto/pr55113_0.c ICEing on x86_64 with -m32 (due to implicit -march=x86-64) by dg-skipping for all x86_64 and the -m64 i?86 variants (untested, not sure what -march default we use there - this just removes incrementally more x86_64 multilibs. Committed. Richard. 2014-03-31 Ric

Re: [PATCH v2] SPARC: Clarify -mapp-regs option

2014-03-31 Thread Richard Biener
On Sun, Mar 16, 2014 at 12:43 PM, Sebastian Huber wrote: > gcc/ChangeLog > 2014-03-16 Sebastian Huber > > * doc/invoke.texi (mapp-regs): Clarify. Ok. Thanks, Richard. > --- > gcc/doc/invoke.texi | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/gcc/doc/inv

RE: Skip gcc.dg/tree-ssa/isolate-*.c for AVR Target

2014-03-31 Thread K_s, Vishnu
Changed the comment in log as the tests are not skipping for AVR only, but for all The targets keeps_null_pointer_checks is true. Corrected the log as per the comments. Please review it and commit it if it's OK. Regards Vishnu KS 2014-03-31 Vishnu K S * gcc/testsuite/gcc.dg/tree-ssa/

Re: [PATCH v2] SPARC: Clarify -mapp-regs option

2014-03-31 Thread Sebastian Huber
Ping. On 2014-03-16 12:43, Sebastian Huber wrote: gcc/ChangeLog 2014-03-16 Sebastian Huber * doc/invoke.texi (mapp-regs): Clarify. --- gcc/doc/invoke.texi | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 24bd76

Re: Changing INT to SI mode

2014-03-31 Thread Richard Biener
On Fri, Mar 28, 2014 at 8:18 PM, Mike Stump wrote: > On Mar 28, 2014, at 6:23 AM, K_s, Vishnu wrote: >> Test pr59940.c is failing for AVR target because the test assumes the size >> of int as 32 bit and test expect to generate warnings for overflow and >> conversion while assigning 36-bit and 32

Re: [C PATCH] Warn if inline attributes conflict (PR c/18079)

2014-03-31 Thread Richard Biener
On Sun, Mar 30, 2014 at 9:25 PM, Richard Sandiford wrote: > Marek Polacek writes: >> @@ -,7 +,16 @@ handle_noinline_attribute (tree *node, tree name, >> int ARG_UNUSED (flags), bool *no_add_attrs) >> { >>if (TREE_CODE (*node) == FUNCTION_DECL) >> -DECL_UN

Re: [PATCH] Fixing PR60656

2014-03-31 Thread Richard Biener
On Sat, Mar 29, 2014 at 2:26 AM, Cong Hou wrote: > This patch is fixing PR60656. Elements in a vector with > vect_used_by_reduction property cannot be reordered if the use chain > with this property does not have the same operation. > > Bootstrapped and tested on a x86-64 machine. > > OK for trunk

Re: [PATH, SH] Small builtin_strlen improvement

2014-03-31 Thread Christian Bruel
On 03/30/2014 11:02 PM, Oleg Endo wrote: > Hi, > > On Wed, 2014-03-26 at 08:58 +0100, Christian Bruel wrote: > >> This patches adds a few instructions to the inlined builtin_strlen to >> unroll the remaining bytes for word-at-a-time loop. This enables to have >> 2 distinct execution paths (no fall

[PATCH, FORTRAN] Fix PR fortran/60191

2014-03-31 Thread Bernd Edlinger
Hi, in the test cases dynamic_dispatch_1.f03 and dynamic_dispatch_3.f04 there is recursive procedure type, make_real, the front-end generates the equivalent of extern "C" float make_real(t1*, ...); for the call statements, but uses extern "C" float make_real(t1*) { } for the procedure itself

RE: Changing INT to SI mode

2014-03-31 Thread K_s, Vishnu
I have found one more test with same issue. 2014-03-31 Vishnu K S * gcc.dg/pr59963-2.c (bazi): Use 32-bit SI mode instead of int. diff --git a/gcc/testsuite/gcc.dg/pr59963-2.c b/gcc/testsuite/gcc.dg/pr59963-2.c index ea05946..263ac37 100644 --- a/gcc/testsuite/gcc.dg/pr59963-2.c +++ b