Re: fix for pr47837

2011-03-14 Thread Richard Guenther
On Sat, Mar 12, 2011 at 1:44 AM, Xinliang David Li davi...@google.com wrote: On Fri, Mar 11, 2011 at 11:50 AM, Jeff Law l...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/09/11 09:24, Xinliang David Li wrote: On Wed, Mar 9, 2011 at 6:03 AM, Jeff Law l...@redhat.com

Add a testcase for reload PR 47166

2011-03-14 Thread Richard Sandiford
A reload patch that I posted in January: http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00548.html turned out to be related to PR 47166, so Bernd committed a combined fix: http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01620.htmlhttp://gcc.gnu.org/ml/gcc-patches/2011-01/msg01620.html Is

Fix for PR target/47951

2011-03-14 Thread Henderson, Stuart
Hi, The attached patch is an attempt to fix PR 47951, however I'm fairly new to GCC and have little confidence in it. I've done some testing with it and it avoids the problem and doesn't appear to cause any new problems, but I'd appreciate more experienced eyes looking over it and giving

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, + /* Determine if the exit test is formulated in terms of the phi or the + increment of the use iv. */ + use_uses_inced_iv += gimple_code (SSA_NAME_DEF_STMT (use-iv-ssa_name)) != GIMPLE_PHI; + + /* Determine if the exit test is before or after the increment of the +

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, it is trying to allow for do { *p = '\0'; i++; /* use_uses_inced_iv == true */ p++; /* use_after_cand_inc == true */ if (!(i n)) break; } while (1); and for do { *p = '\0'; if (!(i n)) break; i++; /*

[cxx-mem-model] options for controlling data races

2011-03-14 Thread Aldy Hernandez
The following patch adds the --param(eters) needed to tweak the optimizers into allowing or disallowing data races in cross-thread visible data. There will be additional flags added later to make these parameters available to the common user. It has been suggested to use -fmemory-model=xxx,

[patch libcpp]: Improve handling of DOS-filenames and -paths

2011-03-14 Thread Kai Tietz
Hello, this patch fixes some DOS-filename comparision and directory-separator checks. Changelog libcpp 2011-03-14 Kai Tietz * files.c (file_hash_eq): Use filename_cmp instead of strcmp. (nonexistent_file_hash_eq): Likewise. (remap_filename): Likewise.

Re: PATCH: Enable __float128 in gcc.target/i386/builtin-copysign.c for 32bit

2011-03-14 Thread Uros Bizjak
On Sat, Mar 12, 2011 at 7:28 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, __float128 has been supported on ia32 for a while.  This patch removes __LP64__ check.  OK for 4.7? Thanks. H.J. --- 2011-03-12  H.J. Lu  hongjiu...@intel.com        * gcc.target/i386/builtin-copysign.c: Remove

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

2011-03-14 Thread Uros Bizjak
Hello! 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 remember the exact details). The patch is

[DOC PATCH] 4.6 branching changes

2011-03-14 Thread Jakub Jelinek
Hi! I've committed these changes to reflect the creation of 4.6 branch. --- c99status.html 6 Apr 2010 11:34:44 - 1.55 +++ c99status.html 14 Mar 2011 13:52:00 - 1.56 @@ -21,6 +21,7 @@ paragraph 6) do not count as library iss pThis page describes the C99 support in

[PATCH] Trunk is now 4.7

2011-03-14 Thread Jakub Jelinek
Hi! 2011-03-14 Jakub Jelinek ja...@redhat.com * BASE-VER: Change to 4.7.0. --- gcc/BASE-VER(revision 170924) +++ gcc/BASE-VER(revision 170945) @@ -1 +1 @@ -4.6.0 +4.7.0 Jakub

Re: Add a testcase for reload PR 47166

2011-03-14 Thread Jakub Jelinek
On Mon, Mar 14, 2011 at 12:34:16PM +0100, Richard Guenther wrote: On Mon, Mar 14, 2011 at 11:25 AM, Richard Sandiford richard.sandif...@linaro.org wrote: A reload patch that I posted in January:    http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00548.html turned out to be related to PR

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Bernd Schmidt
On 03/14/2011 06:10 PM, Richard Earnshaw wrote: On Sun, 2011-03-13 at 16:31 +, Andrew Stubbs wrote: This patch discourages the use of NEON for integer operations on ARM Cortex-A8. The problem is that transferring data from NEON/VFP registers to core registers is prohibitively

Re: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD sizeof (void *)

2011-03-14 Thread H.J. Lu
On Sun, Mar 6, 2011 at 4:15 PM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Mar 6, 2011 at 3:40 PM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Mar 6, 2011 at 3:23 PM, Richard Guenther richard.guent...@gmail.com wrote: On Sun, Mar 6, 2011 at 10:28 PM, H.J. Lu hjl.to...@gmail.com wrote: On Sun,

Re: Patch: suppress -Woverlength-strings in asm()

2011-03-14 Thread Tom Tromey
2011-02-02 Tom Tromey tro...@redhat.com * c-parser.c (c_parser_asm_string_literal): Clear warn_overlength_strings. 2011-02-02 Tom Tromey tro...@redhat.com * gcc.dg/Woverlength-strings-pedantic-c90-asm.c: New file. * gcc.dg/Woverlength-strings-pedantic-c89-asm.c: New file. *

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Andrew Stubbs
On 14/03/11 17:10, Richard Earnshaw wrote: Finally, alternatives from an insn are normally selected left-to-right from those available in a pattern, all other things being equal. So really the A8-only alternative should come after the core registers alternatives if its less preferable. OK, I

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Richard Earnshaw
On Mon, 2011-03-14 at 17:55 +, Andrew Stubbs wrote: On 14/03/11 17:10, Richard Earnshaw wrote: Finally, alternatives from an insn are normally selected left-to-right from those available in a pattern, all other things being equal. So really the A8-only alternative should come after

Re: Patch: add systemtap-style marker to _Unwind_DebugHook

2011-03-14 Thread Tom Tromey
Roland That test would work. (That macro is not defined at all in Roland earlier versions.) Tom Here is an updated version. Tom 2011-01-14 Tom Tromey tro...@redhat.com Tom* unwind-dw2.c: Include sys/sdt.h if it exists. Tom(_Unwind_DebugHook): Use STAP_PROBE2. Tom* config.in,

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-03-14 Thread Rainer Orth
Richard Guenther rguent...@suse.de writes: Can you update your patch with the tri-state solution? Sure if the solution is deemed acceptable. There isn't much point in following that route if you see problems up front. If that solution avoids 3) then yes, I'm fine with going that route.

Re: PATCH: PR target/46770: Use .init_array/.fini_array sections

2011-03-14 Thread H.J. Lu
On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu hongjiu...@intel.com wrote: On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote: This patch uses .init_array/.fini_array sections instead of .ctors/.dtors sections

[v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Rainer Orth
In their infinite wisdom, Oracle has moved all contents from docs.sun.com to the Oracle Technical Network without putting specific redirects in place, thus invalidating more than a decade of links in a single stroke. Oh well ;-( The following patch deals with the outfall. I've decided to remove

PATCH: PR target/48084: [x32] internal compiler error: in copy_to_mode_reg, at explow.c:630

2011-03-14 Thread H.J. Lu
Hi Uros, Is this patch: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00679.html OK for trunk? Thanks. H.J.

Patch ping

2011-03-14 Thread Jakub Jelinek
Hi! http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01749.html - PR middle-end/47917, snprintf folding Jakub

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Rainer Orth
Benjamin Kosnik b...@redhat.com writes: libstdc++-v3: * doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their OTN equivalents. This is ok. Thanks. Jakub, ok for 4.6, too? Unless I'm mistaken, even doc patches require RM approval now? Rainer --

Re: Patch ping

2011-03-14 Thread Diego Novillo
On Mon, Mar 14, 2011 at 16:19, Jakub Jelinek ja...@redhat.com wrote: Hi! http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01749.html  - PR middle-end/47917, snprintf folding OK. Diego.

[patch, fortran] Function call optimization

2011-03-14 Thread Thomas Koenig
Hello world, the attached patch is a front-end optimization which replaces multiple calls to a function with identical argument lists with an assignment to a temporary variable, and then uses that variable in the original expression. AFAIK, this is permitted by the Fortran standard because

[google] XFAIL some guality tests (issue4289045)

2011-03-14 Thread Diego Novillo
These tests fail on trunk as well as google/integration. XFAILing to pacify testers. Tested on x86_64/{-m32,-m64}. Committed to google/integration. 2011-03-14 Diego Novillo dnovi...@google.com * gcc.dg/guality/vla-1.c: XFAIL. * gcc.dg/guality/vla-2.c: XFAIL. *

Ping^2 Re: [PATCH, ARM] Fix PR 43872, incorrectly aligned VLAs

2011-03-14 Thread Chung-Lin Tang
Ping. On 2011/3/7 10:10 PM, Chung-Lin Tang wrote: Ping. On 2011/2/17 06:01 PM, Chung-Lin Tang wrote: Hi, this PR is a case where we have a leaf function with a zero-size frame, that calls alloca() (here through a C99 VLA). The ARM backend recognizes the leaf-and-no-frame opportunity to

Ping^3 Re: [patch, ARM] ICE in get_arm_condition_code()

2011-03-14 Thread Chung-Lin Tang
Ping. On 2011/3/7 10:10 PM, Chung-Lin Tang wrote: Ping. On 2011/1/26 11:09 AM, Chung-Lin Tang wrote: Ping. (I'll fix the minor nits before actually committing, thanks Ramana :) On 2011/1/13 08:24, Chung-Lin Tang wrote: Hi, there's an ICE in arm.c:get_arm_condition_code(), from here:

Re: Fix pr48032, powerpc64 -mcmodel=medium invalid ld offset

2011-03-14 Thread Alan Modra
After some discussion with David and Richard, this is what I'm about to commit. Differs from the previous patch in returning false for BLKmode (via GET_MODE_SIZE(mode)==0 test) in the !decl branch of offsettable_ok_by_alignment. PR target/48032 * config/rs6000/rs6000.c

[PATCH] Re-enable lto on powerpc-apple-darwin*

2011-03-14 Thread Jack Howarth
The assembler bug in Xcode 3.2.6/4.0 can't impact powerpc-apple-darwin* (except in the old-ball case of building the powerpc-apple-darwin* target under Rosetta on Snow Leopard), lto can be re-enabled for that target in gcc trunk. Re-enabling lto for powerpc-apple-darwin* in gcc trunk allows us

[v3] regex versioned namespaces

2011-03-14 Thread Benjamin Kosnik
Ugh. While doing other things I realized that the namespace versioning for regex header files is off: it's not on the inner-most nested namespace. Fixed thusly. tested x86/linux tested x86/linux --enable-symvers=gnu-namespace-versioned -benjamin2011-03-14 Benjamin Kosnik b...@redhat.com *

Re: [DOC PATCH] Add -Wunused-but-set-* note to gcc-4.6/changes.html and mention removal of cstddef in STL headers

2011-03-14 Thread Matthias Klose
On 10.03.2011 17:04, Jakub Jelinek wrote: +default by code-Wall/code flag and code-Wunused-but-set-parameter/code +by code-Wall -W/code flags./li -W is documented as old option. Maybe use -Wextra instead? Matthias

Re: [v3] regex versioned namespaces

2011-03-14 Thread Hans-Peter Nilsson
On Mon, 14 Mar 2011, Benjamin Kosnik wrote: Ugh. While doing other things I realized that the namespace versioning for regex header files is off: it's not on the inner-most nested namespace. Fixed thusly. But it seems like it's not committed? brgds, H-P

Re: [v3] typeinfo tuning

2011-03-14 Thread Benjamin Kosnik
It broke build, hopefully for trivial reasons. It seems like your bugzilla options are set to not receive email, so I'll have to bug you like this, and ask that you please have a look at PR48130. No problem. Sorry about the breakage. Indeed, it was trivial. tested x86_64/linux tested