[PATCH] libdecnumber: start a gitignore

2011-03-17 Thread Mike Frysinger
Signed-off-by: Mike Frysinger vap...@gentoo.org --- libdecnumber/.gitignore |1 + 1 files changed, 1 insertions(+), 0 deletions(-) create mode 100644 libdecnumber/.gitignore diff --git a/libdecnumber/.gitignore b/libdecnumber/.gitignore new file mode 100644 index 000..7e47161 ---

Re: [wwwdocs] gcc-4.6/porting_to.html

2011-03-17 Thread Hans-Peter Nilsson
On Wed, 16 Mar 2011, Benjamin Kosnik wrote: Needs some more work, here's a rough draft. s/Porting to the new tools/Porting to the new version/ ? brgds, H-P

Re: [patch i386,c,c++]: PR/12171 - calling convention omitted in error message

2011-03-17 Thread Kai Tietz
PING, ok for 4.7? 2011/1/4 Jason Merrill ja...@redhat.com: On 01/01/2011 01:07 PM, Kai Tietz wrote: Well, as here no further agreement was found, I post here the alternative suggested by Joseph. I am open-minded which approach will be chosen. I just want to fix this long pending issue. I

[libgfortran, 4.6] Don't use clock_gettime on Tru64 UNIX (PR fortran/47571)

2011-03-17 Thread Rainer Orth
The conditional use of clock_gettime via weakrefs completely broke Fortran on Tru64 UNIX: while the function is available in librt, the platform doesn't support weakrefs, so every execution test fails due to the undefined symbol. While a proper patch is being tested for mainline (to be backported

[doc] Update install.texi for Solaris 2, IRIX, Tru64 UNIX

2011-03-17 Thread Rainer Orth
I've just reviewed install.texi for my platforms and found it mostly accurate. A few minor changes are desirable for the 4.6 release, but also apply to mainline since the file is identical there: * Update recommendations to binutils 2.21 since this is what I've been testing. * Remove download

Re: [PATCH][i386] Implement ix86_emit_swdivsf more efficiently

2011-03-17 Thread Michael Matz
Hi, On Mon, 14 Mar 2011, Richard Guenther wrote: This rewrites the iteration step of swdivsf to be more register efficient (two registers instead of four, no load of a FP constant). This matches how ICC emits the rcp sequence and causes no overall loss of precision (Micha might still

Re: [patch] Enhance conditional store sinking

2011-03-17 Thread Richard Guenther
On Wed, Mar 16, 2011 at 12:47 PM, Ira Rosen ira.ro...@linaro.org wrote: On 16 March 2011 12:29, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Mar 16, 2011 at 7:49 AM, Ira Rosen ira.ro...@linaro.org wrote: Hi, This patch adds a support of conditional store sinking for cases with

[PATCH] Fix PR48134

2011-03-17 Thread Richard Guenther
This fixes PR48134, when propagating into debug-stmts we should fold them, like we do elsewhere. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-03-17 Richard Guenther rguent...@suse.de PR middle-end/48134 * tree-ssa.c

Re: [patch] Enhance conditional store sinking

2011-03-17 Thread Richard Guenther
On Thu, Mar 17, 2011 at 3:53 PM, Jeff Law l...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/17/11 08:48, Richard Guenther wrote: The gimple_uids are not initialized here, you need to make sure to call renumber_gimple_stmt_uids () before starting.  Note that phiopt

[PATCH][RFC] Unify constant folding

2011-03-17 Thread Richard Guenther
This tries to move us towards a single constant-folding machinery on gimple, usable by the various value-numbering passes we have (CCP, VRP, SCCVN and eventually DOM, not yet in this patch). At least VRP and SCCVN should in theory perform the same constant propagations as CCP does but they do

Re: [PATCH] PR/Target 47998

2011-03-17 Thread Yoshinori Sato
At Wed, 16 Mar 2011 10:22:20 -0600, Jeff Law wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/16/11 09:32, Yoshinori Sato wrote: Hi All, This problem optimize rule missing. gen_lowpart got invalid operand. I attached fix patch. diff --git a/gcc/ChangeLog

Re: RX: 4.5 branch: Fix alignment and addressing issues.

2011-03-17 Thread Richard Henderson
On 03/17/2011 05:31 AM, Nick Clifton wrote: So really I think that I should be defining the ADJUST_INSN_LENGTH macro to handle these patterns. I'll look into that. What is length used for in the rx port? I don't see any branch shortening going on here; out of range branches are completely

Re: C++ PATCH for c++/48089 (ICE with invalid constexpr ctor)

2011-03-17 Thread Jason Merrill
On 03/16/2011 11:44 PM, Gabriel Dos Reis wrote: I am not sure we need more infrastructure or more complexity in the implementation. The (C++98) language already requires us to initialize subobjects in their order of declaration. That is what we do here. All we need is to check that a member,

[Patch] Disable -fstack-limit for bfin -mfdpic

2011-03-17 Thread Henderson, Stuart
-fstack-limit- is not supported for bfin with -mfdpic and crashes if you try to use it. The attached patch warns the user that -fstack-limit is ignored with the -mfdpic switch for Blackfin and then disables it. 2011-03-17 Stuart Henderson stuart.hender...@analog.com Originally From Bernd

Re: PR45273 - The compiler depends on the host double

2011-03-17 Thread Richard Henderson
On 03/17/2011 01:35 AM, Richard Guenther wrote: Anyone a better suggestion for PRED_PRECISION? It at least seems you are introducing another host dependency here with using HOST_BITS_PER_WIDE_INT. Why not simply use a fixed value of 32? Or 64, given that's the exact precision that sreal

C++ PATCH for c++/48166 (ICE with invalid function-cv-quals)

2011-03-17 Thread Jason Merrill
I missed one case where we needed to change cp_type_quals to type_memfn_quals. After that fix we gave the correct error, but still ICEd, so to avoid that I go ahead and strip the offending cv-quals. Tested x86_64-pc-linux-gnu, applying to trunk, will apply to 4.6.1 after 4.6.0 release.

Re: PR debug/47510

2011-03-17 Thread Mark Mitchell
On 3/17/2011 4:08 AM, Dodji Seketeli wrote: Yesterday after discussing this on IRC, Jakub expressed his personal opinion by saying the patch could go in 4.6. I mistakenly took it as a formal approval from the RMs and I committed it. I should have waited for an approval by email. You don't

Re: [patch i386,c,c++]: PR/12171 - calling convention omitted in error message

2011-03-17 Thread Kai Tietz
2011/3/17 Jason Merrill ja...@redhat.com: On 03/17/2011 04:46 AM, Kai Tietz wrote: PING, ok for 4.7? Did you have a response to my comment below? 2011/1/4 Jason Merrillja...@redhat.com: On 01/01/2011 01:07 PM, Kai Tietz wrote: Well, as here no further agreement was found, I post here

Re: [patch i386,c,c++]: PR/12171 - calling convention omitted in error message

2011-03-17 Thread Kai Tietz
2011/3/17 Jason Merrill ja...@redhat.com: On 03/17/2011 12:08 PM, Kai Tietz wrote: I thought to use here instead of on_diagnostic (which is IMHO fine too as it indicates for now only that attribute shall be displayed on diagnostics) affects_abi. I think it makes sense to keep that name as

Re: [AVR] Hookize LIBCALL_VALUE and FUNCTION_VALUE_REGNO_P

2011-03-17 Thread Richard Henderson
On 03/16/2011 02:46 PM, Georg-Johann Lay wrote: Anatoly Sokolov schrieb: /* Returns register number for function return value.*/ -int +static inline int avr_ret_register (void) { return 24; } I always wondered why that works. SI is returned in r22..r25 (not in r24..27)

Re: RX: 4.5 branch: Fix alignment and addressing issues.

2011-03-17 Thread DJ Delorie
What is length used for in the rx port? We have a local patch that uses the length to decide if/when to align labels; it goes along with the label alignment change I made a while back. However, the patch works best in 4.5 (align patch not backported) and there are other optimization problems

Re: PR debug/47510

2011-03-17 Thread Dodji Seketeli
Mark Mitchell m...@codesourcery.com writes: And, in any case, if it's a regression it's OK with me. Thanks. I have committed the patch back into 4.6. -- Dodji

Re: [PATCH] PR/Target 47998

2011-03-17 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/17/11 09:31, Yoshinori Sato wrote: At Wed, 16 Mar 2011 10:22:20 -0600, Jeff Law wrote: On 03/16/11 09:32, Yoshinori Sato wrote: Hi All, This problem optimize rule missing. gen_lowpart got invalid operand. I attached fix patch. diff

[Ada] Fix ICE on declaration of discriminated record type

2011-03-17 Thread Eric Botcazou
This is an internal error in self_referential_size on the code generated for the elaboration of a discriminated record type which contains an array whose upper bound depends on the discriminant and whose nominal subtype is an unconstrained array type whose index type has a component of a

Re: [PATCH] Fix PR48148

2011-03-17 Thread H.J. Lu
On Thu, Mar 17, 2011 at 6:24 AM, Richard Guenther rguent...@suse.de wrote: This fixes PR48148, we shouldn't keep abstract origin references for external decls. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-03-17  Richard Guenther  rguent...@suse.de        

[testsuite] Remove explicit -O2 from gcc.c-torture/compile/pr44686.c

2011-03-17 Thread Martin Jambor
Hi, I'm about to check in the following as obvious. The explicit -O2 defeats the different O levels the torture mechanism tries. Tested on x86_64-linux by running make -k check RUNTESTFLAGS=compile.exp=pr44686.c and verifying all tests passed. Thanks, Martin Index:

Re: C++ PATCH for c++/48089 (ICE with invalid constexpr ctor)

2011-03-17 Thread Gabriel Dos Reis
On Thu, Mar 17, 2011 at 10:41 AM, Jason Merrill ja...@redhat.com wrote: On 03/16/2011 11:44 PM, Gabriel Dos Reis wrote: I am not sure we need more infrastructure or more complexity in the implementation.  The (C++98) language already requires us to initialize subobjects in their order of

PATCH: PR target/48171: Missing Save on -mavx and -mfma

2011-03-17 Thread H.J. Lu
Hi, All i386 ISA options, except for -mavx and -mfma, have Save. This patch fixes it. OK for trunk and 4.6? Thanks. H.J. --- 2011-03-17 H.J. Lu hongjiu...@intel.com PR target/48171 * config/i386/i386.opt: Add Save to -mavx and -mfma. diff --git a/gcc/config/i386/i386.opt

Re: RX: 4.5 branch: Fix alignment and addressing issues.

2011-03-17 Thread Mike Stump
On Mar 17, 2011, at 5:31 AM, Nick Clifton wrote: You can set up the correct length even without using separate patterns. The problem is And why is that a problem? You get to write arbitrarily complex C code that can depend upon insn and operands. I presume that you can do that. See:

Re: [PATCH] Adjust DECL_INCOMING_RTL for pass_by_reference parameters (PR debug/48163)

2011-03-17 Thread Richard Henderson
On 03/17/2011 11:36 AM, Jakub Jelinek wrote: PR debug/48163 * function.c (assign_parms): For data.passed_pointer parms use MEM of data.entry_parm instead of data.entry_parm itself as DECL_INCOMING_RTL. * dwarf2out.c (rtl_for_decl_location): Use DECL_INCOMING_RTL

Re: [PATCH] Fix handling of calls to constants (PR debug/48163)

2011-03-17 Thread Richard Henderson
On 03/17/2011 11:40 AM, Jakub Jelinek wrote: PR debug/48163 * var-tracking.c (prepare_call_arguments): If CALL target is a non-SYMBOL_REF CONSTANT_P, just add that into the list as pc instead of looking it up using cselib_lookup and use Pmode for it if x has

Re: [PATCH] Fix hppa-* --enable-checking=release bootstrap (PR bootstrap/48161)

2011-03-17 Thread Jakub Jelinek
On Thu, Mar 17, 2011 at 12:54:44PM -0700, Richard Henderson wrote: On 03/17/2011 11:46 AM, Jakub Jelinek wrote: - result = gen_rtx_PLUS (tmode, result, tmp); + { + if (swap_commutative_operands_p (result, tmp)) + result = gen_rtx_PLUS (tmode, tmp, result); + else +

Reinstalled patch for 37273

2011-03-17 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I had pulled the patch for 37273 during the 4.6 cycle due to exposing several latent problems. I've just reinstalled it and will (of course) keep an eye out for any problems. -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux)

Fix problem with guality tests

2011-03-17 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 The guality tests can randomly fail due to expect buffering issues. Given this output: Breakpoint 1 at 0x4004d6: file /home/gcc/virgin-gcc/gcc/testsuite/gcc.dg/guality/pr43479.c, line 18.^M ^M Breakpoint 1, foo (k=7, l=Unhandled dwarf expression

C++ PATCH for c++/47504 (bogus overflow error with constexpr)

2011-03-17 Thread Jason Merrill
In this testcase, the fix for PR 25125 causes us to rewrite what starts as char(int(-1) - int(1)) to char((unsigned char)(-1) - (unsigned char)1) and thus char((unsigned char)254). 254 doesn't fit in char, so the result has TREE_OVERFLOW set even though the original expression was all signed

Re: [libgfortran, patch] More than 26 temporary files with weak mktemp()

2011-03-17 Thread FX
Thanks for the review! - Use the type size_t for tempdirlen as that is the return type of strlen() and argument type for get_mem(). - You can use a const size_t variable for the length of the string slash rather than calling strlen() in the do-while loop. Both OK. - Don't set errno as we

Can't use SImode as Pmode for x32

2011-03-17 Thread H.J. Lu
On Wed, Mar 16, 2011 at 4:28 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Mar 16, 2011 at 03:38:25PM -0700, H.J. Lu wrote: IMNSHO you really should reconsider using Pmode != ptr_mode for your port. Hi Jakub, I created hjl/x32/Pmode/master branch at

Re: [PATCH] Fix hppa-* --enable-checking=release bootstrap (PR bootstrap/48161)

2011-03-17 Thread Jakub Jelinek
On Thu, Mar 17, 2011 at 09:07:43PM +0100, Jakub Jelinek wrote: The following fixes the testcase in cross to hppa-* as well. I'll bootstrap/regtest it momentarily, ok if it passes? Now bootstrapped/regtested on x86_64-linux and i686-linux. 2011-03-17 Richard Henderson r...@redhat.com

Re: PATCH: PR rtl-optimization/47502: Never combine asm statement

2011-03-17 Thread H.J. Lu
Hi Eric, Is this patch OK for trunk? Thanks. H.J. --- On Thu, Jan 27, 2011 at 6:14 PM, H.J. Lu hongjiu...@intel.com wrote: It is a bad idea to combine asm statement.  This patch disallows it. Any comments? Thanks. H.J. --- commit a11b95180e53d3a0bc3eabad5594859cea7c3334 Author: H.J.

Re: PATCH: PR rtl-optimization/47502: Never combine asm statement

2011-03-17 Thread Geert Bosch
On Mar 17, 2011, at 20:35, H.J. Lu wrote: - substitutions of likely-spilled regs, reload might die. + substitutions of likely-spilled regs, reload might die. Never + combine asm statement. This has to be statements, a plural. -Geert

Re: PATCH: PR rtl-optimization/47502: Never combine asm statement

2011-03-17 Thread H.J. Lu
On Thu, Mar 17, 2011 at 8:28 PM, Geert Bosch bo...@adacore.com wrote: On Mar 17, 2011, at 20:35, H.J. Lu wrote: -     substitutions of likely-spilled regs, reload might die. +     substitutions of likely-spilled regs, reload might die.  Never +     combine asm statement. This has to be

Re: [x32] PATCH: PR middle-end/47725: [x32] error: unable to find a register to spill in class DIREG

2011-03-17 Thread H.J. Lu
On Thu, Mar 17, 2011 at 5:30 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, Feb 15, 2011 at 2:55 PM, Bernd Schmidt ber...@codesourcery.com wrote: On 02/14/2011 08:46 PM, Eric Botcazou wrote: I agree with Jeff that combine would be the correct place to fix this. At least it takes