[PATCH] fix for c/PR57577

2013-06-12 Thread Iyer, Balaji V
Hello Everyone, This patch fixes an ICE when rank mismatch occurs in a modify expression where the right-hand side is a function call. The crash is actually occuring in a while loop whose result is not used, i.e. dead code. I removed the dead code and things seem to work fine now. Teste

Re: [PATCH 1/4] Fix trailing whitespaces

2013-06-12 Thread Joseph S. Myers
In general I think no formatting fixes should be made to GCC testcases, including removal of trailing whitespace; it's good if they cover a range of coding styles and oddities as that reflects how GCC is used in practice. -- Joseph S. Myers jos...@codesourcery.com

Go patch committed: Implement Go 1.1 terminating statements

2013-06-12 Thread Ian Lance Taylor
The Go 1.1 release defines what it means for a function to have a terminating statement, and makes it an error for a function with return values to not have one. This patch, from Rémy Oudompheng, implements that for gccgo. Bootstrapped and ran testsuite on x86_64-unknown-linux-gnu. Committed to

[google/gcc-4_8] Add support for .debug_gnu_pubnames/pubtypes section.

2013-06-12 Thread Cary Coutant
This patch is ported from the google/gcc-4_7 branch. Tested with crosstool_validate.py and committed. -cary include: 2012-08-20 Sterling Augustine * gdb/gdb-index.h: Check in from src. gcc: 2012-08-20 Sterling Augustine * dwarf2out.c (DEBUG_PUBNAMES_SECTION, DEBUG_PUB

Re: [PATCH] PR57518, RA generated redundent code

2013-06-12 Thread H.J. Lu
On Wed, Jun 12, 2013 at 2:44 PM, Wei Mi wrote: > Hi, > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57518 > > pr57518 happened because update_equiv_regs in IRA marked a reg > equivalent with a mem, lowered its mem_cost in scan_one_insn, set > NO_REGS to its rclass, but didn't consider the reg was

Re: [Bug libstdc++/56430] In __airy: return-statement with a value, in function returning 'void'.

2013-06-12 Thread Ed Smith-Rowland
Original Message Subject: Re: [Bug libstdc++/56430] In __airy: return-statement with a value, in function returning 'void'. Date: Wed, 12 Jun 2013 20:02:27 -0400 From: Ed Smith-Rowland <3dw...@verizon.net> To: libstd...@gcc.gnu.org , gcc-patches Origi

Re: C++: fix ICE with CONST_DECLs

2013-06-12 Thread Mike Stump
On Jun 11, 2013, at 11:00 AM, Jason Merrill wrote: > On 06/11/2013 12:39 PM, Paolo Carlini wrote: >> On 06/11/2013 06:27 PM, Mike Stump wrote: >>> Here is a simple one. When processing CONST_DECLs after an error, we >>> can ICE. This avoid the ICE. > >> No testcase? > > Yep; the patch is fine,

Re: C++: fix ICE with CONST_DECLs

2013-06-12 Thread Jason Merrill
Go ahead and apply the patch; it never hurts to make the code more robust. Jason

Re: [Bug libstdc++/56430] In __airy: return-statement with a value, in function returning 'void'.

2013-06-12 Thread Paolo Carlini
On 06/13/2013 02:10 AM, Ed Smith-Rowland wrote: The point of the patch is to fix the base function and then offer access to the function as a gcc extension. boost has these functions. These have been offered in C++ library papers extending TR 29123. If Boost already has those extensions and are

[C++ Patch] PR 57599

2013-06-12 Thread Paolo Carlini
Hi, this dynamic_cast bug, which just came in, notices that in the simple case of static conversion we forget to perform the cv-qualifier conversions. It seems to me that we are missing a build_nop. I extended quite a bit the testcase, to references too, because at some point I got those wro

Re: GCC does not support *mmintrin.h with function specific opts

2013-06-12 Thread Sriraman Tallam
Hi Honza, I have isolated the ipa-inline.c part into a separate patch with a test and attached it here. The patch is simple. Could you please take a look? * ipa-inline.c (can_early_inline_edge_p): Flag an error when the function that cannot be inlined is target specific.

Re: [RS6000] IBM long double little-endian

2013-06-12 Thread David Edelsohn
On Wed, Jun 12, 2013 at 10:53 AM, Alan Modra wrote: > FLOAT_WORDS_BIG_ENDIAN doesn't work out too well for IBM extended > double when little-endian, because we're thinking to keep the large > magnitude double first. See the comment below on > LONG_DOUBLE_LARGE_FIRST. > > This patch fixes all occu

[PATCH GCC]Consider NOP_EXPR and CONVERT_EXPR as equal nodes in operand_equal_p

2013-06-12 Thread Bin Cheng
Hi, This is a case of NOP_EXPR and CONVERT_EXPR not compared equal in operand_equal_p, resulting in below two nodes are considered different: NODE 0: unit size align 16 symtab 0 alias set 4 canonical type 0xb74602a0 precision 16 min max context pointer_to_this > a

[PATCH GCC]Check the code to be executed for COND_EXEC in noop_move_p

2013-06-12 Thread Bin Cheng
Hi, For arm, gcc generates redundant assembly code like "movlt r4, r4", because it doesn't check noop move for COND_EXEC instructions. This patch fixes the issue by checking the inner code to be executed for COND_EXEC in noop_move_p. Bootstrap and test on both x86 and cortex-a15. Is it OK? Than

Re: [RFC] Inconsistency in ordering vector widening operations on big-endian targets?

2013-06-12 Thread Alan Modra
On Wed, Jun 12, 2013 at 04:22:46PM +0100, Tejas Belagod wrote: > From the IR above, it seems that result of widening the high part > (vect__12.6_40) is being stored at offset 0 from _29 and result of > widening the low part goes into *(_29 + 16). Shouldn't this be the > other way around? Big-endia

powerpc64le abi check

2013-06-12 Thread Alan Modra
In order to check powerpc64le abi symbols. Bootstrapped etc. powerpc64-linux. OK to apply? * configure.host (abi_baseline_pair): Match powerpc64*. Index: libstdc++-v3/configure.host === --- libstdc++-v3/configure.host (revi

Re: [PATCH] PR57518, RA generated redundent code

2013-06-12 Thread Wei Mi
The testcase is attached. Thanks, Wei. On Wed, Jun 12, 2013 at 5:03 PM, H.J. Lu wrote: > On Wed, Jun 12, 2013 at 2:44 PM, Wei Mi wrote: >> Hi, >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57518 >> >> pr57518 happened because update_equiv_regs in IRA marked a reg >> equivalent with a mem, l

Re: [patch] set MULTIARCH_DIRNAME for multilib architectures

2013-06-12 Thread Bernhard Reutner-Fischer
On 12 June 2013 20:20:50 Richard Sandiford wrote: Matthias Klose writes: > Index: config/mips/t-linux64 > === > --- config/mips/t-linux64 (revision 200012) > +++ config/mips/t-linux64 (working copy) > @@ -24,3 +24,13 @@ > ../li

Re: More forwprop for vectors

2013-06-12 Thread Marc Glisse
On Wed, 12 Jun 2013, Jeff Law wrote: 2013-06-13 Marc Glisse * tree-ssa-forwprop.c (simplify_bitwise_binary, associate_plusminus): Generalize to complex and vector. * tree.c (build_all_ones_cst): New function. * tree.h (build_all_ones_cst): Declare it. This is OK. Extra

<    1   2