[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 vishnu@atmel.com * 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 ---

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-thru in

Re: [PATCH] Fixing PR60656

2014-03-31 Thread Richard Biener
On Sat, Mar 29, 2014 at 2:26 AM, Cong Hou co...@google.com 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

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 rdsandif...@googlemail.com wrote: Marek Polacek pola...@redhat.com writes: @@ -,7 +,16 @@ handle_noinline_attribute (tree *node, tree name, int ARG_UNUSED (flags), bool *no_add_attrs) { if (TREE_CODE

Re: Changing INT to SI mode

2014-03-31 Thread Richard Biener
On Fri, Mar 28, 2014 at 8:18 PM, Mike Stump mikest...@comcast.net wrote: On Mar 28, 2014, at 6:23 AM, K_s, Vishnu vishnu@atmel.com 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

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 sebastian.hu...@embedded-brains.de * 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

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 vishnu@atmel.com *

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 sebastian.hu...@embedded-brains.de wrote: gcc/ChangeLog 2014-03-16 Sebastian Huber sebastian.hu...@embedded-brains.de * doc/invoke.texi (mapp-regs): Clarify. Ok. Thanks, Richard. --- gcc/doc/invoke.texi | 5 - 1 file

[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

[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

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 add the

[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

Re: std::rethrow_exception is broken

2014-03-31 Thread Rainer Orth
Jonathan Wakely jwak...@redhat.com 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 jwak...@redhat.com Date: Tue Mar 25 14:42:45 2014 + PR

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:

[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

[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

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:

[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: [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 sebastian.hu...@embedded-brains.de wrote: gcc/ChangeLog 2014-03-16 Sebastian Hubersebastian.hu...@embedded-brains.de * doc/invoke.texi (mapp-regs): Clarify. Ok. Thanks, Richard. I don't

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

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 vmaka...@redhat.com PR rtl-optimization/60650 * gcc.target/arm/pr60650.c: New. The test fails for me with: spawn -ignore SIGHUP

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 hjl.to...@gmail.com wrote: On Saturday, March 29, 2014, Jakub Jelinek ja...@redhat.com 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

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

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 jwak...@redhat.com: 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

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

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

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 rguent...@suse.de wrote: 2014-03-31 Richard Biener rguent...@suse.de libiberty/ * simple-object.c (simple_object_internal_write): Handle EINTR and short writes. This is OK. Thanks. Ian

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, 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: Lost __mips_o32 predefine on NetBSD

2014-03-31 Thread Richard Sandiford
Martin Husemann mar...@duskware.de 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

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

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

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

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

2014-03-31 Thread Fabien Chêne
2014-03-28 17:59 GMT+01:00 Jason Merrill ja...@redhat.com: 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

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

2014-03-31 Thread Fabien Chêne
2014-03-28 17:59 GMT+01:00 Jason Merrill ja...@redhat.com: [...] 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,

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))

[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

[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