Re: Fix compute_reloc_for_constant

2014-01-19 Thread Bernhard Reutner-Fischer
On 19 January 2014 03:12:56 Jan Hubicka hubi...@ucw.cz wrote: Hi, while comparing LTO and non-LTO builds I noticed that with LTO we produce a lot more vtables in datal.rel.ro rather than data.rel.ro.local This is because of partitioning promoting more symbols global. For RTL we make section

[PATCH] libgcc: use linker script for libgcc_s.so on xtensa

2014-01-19 Thread Baruch Siach
The xtensa port uses __xtensa_libgcc_window_spill in libgcc to implement __builtin_frame_address. This symbol is local/hidden in libgcc. This is not a problem when linking against static libgcc. But g++ defaults to -shared-libgcc, thus breaking link against C++ shared libraries that are using

Re: PATCH: PR target/59379: [4.9 Regression] gomp_init_num_threads is compiled into an infinite loop with --with-arch=corei7 --with-cpu=slm

2014-01-19 Thread Uros Bizjak
On Sat, Jan 18, 2014 at 9:15 PM, H.J. Lu hjl.to...@gmail.com wrote: For LEA operation with SImode_address_operand, which zero-extends SImode to DImode, ix86_split_lea_for_addr turns (set (reg:DI) ...) into (set (reg:SI) ...) We need to do (set (reg:DI) (zero_extend:DI (reg:SI))) at

[committed] Fix vect_intness in a few tests

2014-01-19 Thread Richard Sandiford
One test tests for integer vectorisation so requires vect_int. Two others already had the dg-require-effective-target, but it was before the dg-do rather than after. Tested on mips64-linux-gnu, where it fixes the vect.exp failures. Applied as obvious. Thanks, Richard gcc/testsuite/ *

Re: [Patch] Fix regex multiple consecutive quantifiers bug.

2014-01-19 Thread Paolo Carlini
Hi, On 01/19/2014 06:52 AM, Tim Shen wrote: Regex like a** will throw an unexpected exception. Now fixed (but currently no optimizations on it). Booted and tested with -m64 and -m32 respectively. Ok. Please remove 2013 as copyright year for the tescase. I also think you can avoid the

Re: Fix compute_reloc_for_constant

2014-01-19 Thread Richard Biener
On Sun, Jan 19, 2014 at 3:12 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, while comparing LTO and non-LTO builds I noticed that with LTO we produce a lot more vtables in datal.rel.ro rather than data.rel.ro.local This is because of partitioning promoting more symbols global. For RTL we make

Re: PATCH: PR target/59379: [4.9 Regression] gomp_init_num_threads is compiled into an infinite loop with --with-arch=corei7 --with-cpu=slm

2014-01-19 Thread H.J. Lu
On Sun, Jan 19, 2014 at 1:55 AM, Uros Bizjak ubiz...@gmail.com wrote: On Sat, Jan 18, 2014 at 9:15 PM, H.J. Lu hjl.to...@gmail.com wrote: For LEA operation with SImode_address_operand, which zero-extends SImode to DImode, ix86_split_lea_for_addr turns (set (reg:DI) ...) into (set (reg:SI)

[Patch, libgfortran] PR59771, PR59774, and PR59836 Bugs in FORMATs Fw.0, Gw.0, and Gw.d

2014-01-19 Thread Dominique d'Humières
The attached patch fixes these bugs and adds the tests. See the PRs for = the rationale of the changes. Regression tested on x86_64-apple-darwin13 and powerpc-apple-darwin9. OK for trunk, 4.8.3, and 4.7.4 (after testing)? Regards, Dominique CL Description: Binary data patch-59774t

Re: PATCH: PR target/59379: [4.9 Regression] gomp_init_num_threads is compiled into an infinite loop with --with-arch=corei7 --with-cpu=slm

2014-01-19 Thread Uros Bizjak
On Sun, Jan 19, 2014 at 3:20 PM, H.J. Lu hjl.to...@gmail.com wrote: For LEA operation with SImode_address_operand, which zero-extends SImode to DImode, ix86_split_lea_for_addr turns (set (reg:DI) ...) into (set (reg:SI) ...) We need to do (set (reg:DI) (zero_extend:DI (reg:SI))) at

Re: PATCH: PR target/59379: [4.9 Regression] gomp_init_num_threads is compiled into an infinite loop with --with-arch=corei7 --with-cpu=slm

2014-01-19 Thread H.J. Lu
On Sun, Jan 19, 2014 at 6:24 AM, Uros Bizjak ubiz...@gmail.com wrote: On Sun, Jan 19, 2014 at 3:20 PM, H.J. Lu hjl.to...@gmail.com wrote: For LEA operation with SImode_address_operand, which zero-extends SImode to DImode, ix86_split_lea_for_addr turns (set (reg:DI) ...) into (set

Re: PATCH: PR target/59379: [4.9 Regression] gomp_init_num_threads is compiled into an infinite loop with --with-arch=corei7 --with-cpu=slm

2014-01-19 Thread Uros Bizjak
On Sun, Jan 19, 2014 at 3:30 PM, H.J. Lu hjl.to...@gmail.com wrote: For LEA operation with SImode_address_operand, which zero-extends SImode to DImode, ix86_split_lea_for_addr turns (set (reg:DI) ...) into (set (reg:SI) ...) We need to do (set (reg:DI) (zero_extend:DI (reg:SI))) at

[MIPS, committed] Add -ffat-lto-objects to pr54240.c

2014-01-19 Thread Richard Sandiford
Add -ffat-lto-objects to pr54240.c to fix an UNRESOLVED on the scan-tree-dump. Tested on mips64-linux-gnu and applied. There's a similar failure for octeon2-pipe-1.c but I'm going to fix that in a different way. Thanks, Richard gcc/testsuite/ * gcc.target/mips/pr54240.c: Add

[testsuite] Require -ffat-lto-objects for scan-rtl-dump*

2014-01-19 Thread Richard Sandiford
We don't compile to RTL or machine instructions for lean LTO objects, so the scan-assembler tests force -ffat-lto-objects to be used instead. This patch extends that to scan-rtl-dump*. Tested on mips64-linux-gnu, where it fixes an UNRESOLVED on gcc.target/mips/octeon2-pipe-1.c. OK to install?

PR 52125: Detecting which operands are used in an asm

2014-01-19 Thread Richard Sandiford
The MIPS %hi and %lo relocation operators act as a pair on REL targets; you need to see the partnering %lo in order to calculate the addend for a %hi and detect carries properly. The assembler therefore complains if %hi is used without a corresponding %lo. The MIPS backend tries to remove

Re: Ping: [PATCH] Prevent crash when expand movstr fail when movstr pattern is defined

2014-01-19 Thread Richard Sandiford
Jakub Jelinek ja...@redhat.com writes: On Sat, Jan 18, 2014 at 10:37:18AM +, Richard Sandiford wrote: Review ping for this patch. IMO it almost counts as obviously correct but just in case... Kito Cheng k...@0xlab.org writes: expand_movstr is work fine when we don't define movstr

[PATCH][RFC] Fix PR59860

2014-01-19 Thread Richard Biener
The following patch fixes PR59860 by removing the only folding builtins.c does that can end up recursing to GIMPLE call stmt folding. It does that via strcat - strlen + strcpy folding and then folding the strlen gimple stmt via gimplify which then can use the SSA web to fold that to a constant

Re: [PATCH][RFC] Fix PR59860

2014-01-19 Thread Jakub Jelinek
On Sun, Jan 19, 2014 at 07:05:12PM +0100, Richard Biener wrote: The following patch fixes PR59860 by removing the only folding builtins.c does that can end up recursing to GIMPLE call stmt folding. It does that via strcat - strlen + strcpy folding and then folding the strlen gimple stmt via

[committed] Fix length of long non-pic millicode calls on PA

2014-01-19 Thread John David Anglin
The attached change fixes a segmentation fault bootstrapping GCC on hppa2.0-hp-hpux11.11. The length of long non-pic millicode calls does not depend on TARGET_LONG_ABS_CALL. As a result, the length of long non-pic millicode calls was being overestimated. This broke the millicode call to

[PATCH, testsuite]: g++.dg/pr49718.C: Add -mno-explicit-relocs for alpha*-*-* targets.

2014-01-19 Thread Uros Bizjak
Hello! Alpha needs -mno-explicit-relocs to count number of function calls properly. 2014-01-19 Uros Bizjak ubiz...@gmail.com * g++.dg/pr49718.C: Add -mno-explicit-relocs for alpha*-*-* targets. Tested on alphaev6-pc-linux-gnu and committed to mainline SVN. Uros. Index: g++.dg/pr49718.C

[PING]Resolve pr44194-1.c failure by refining scan-rtl-dump-not pattern

2014-01-19 Thread Renlin Li
On 10/01/14 11:28, Renlin Li wrote: Hi Eric, Thank you for your suggestion! I have made the adjustment and test it. Please check the new attachment. Regards, Renlin Li On 10/01/14 10:42, Eric Botcazou wrote: This patch will resolve testsuite/gcc.dg/pr44194-1.c failure observed in aarch64

Re: [Patch, libgfortran] PR59771, PR59774, and PR59836 Bugs in FORMATs Fw.0, Gw.0, and Gw.d

2014-01-19 Thread Jerry DeLisle
On 01/19/2014 06:24 AM, Dominique d'Humières wrote: The attached patch fixes these bugs and adds the tests. See the PRs for = the rationale of the changes. Regression tested on x86_64-apple-darwin13 and powerpc-apple-darwin9. OK for trunk, 4.8.3, and 4.7.4 (after testing)? Regards,

Re: [Patch, libgfortran] PR59771, PR59774, and PR59836 Bugs in FORMATs Fw.0, Gw.0, and Gw.d

2014-01-19 Thread Dominique d'Humières
Yes OK, and I will commit for you. Thanks Jerry. While trying to back port the patch, I have found a leftover of my previous attempts: if (ft != FMT_F before == 0 w 0 d == 0 p == 0) should be if (ft != FMT_F w 0 d == 0 p == 0) The second change was needed for 4.7 (still one

Re: experimental testsuite patch

2014-01-19 Thread François Dumont
So here is another patch proposal with the faulty debug assertion commented for the moment. 2014-01-20 François Dumont fdum...@gcc.gnu.org * scripts/create_testsuite_files: Add testsuite/experimental in the list of folders to introspect for tests. *

Re: [Patch] Fix regex multiple consecutive quantifiers bug.

2014-01-19 Thread Tim Shen
On Sun, Jan 19, 2014 at 5:01 AM, Paolo Carlini paolo.carl...@oracle.com wrote: Ok. Please remove 2013 as copyright year for the tescase. I also think you can avoid the auxiliary includes. Oops, fixed, including fixing incorrect file name :) Tested and committed. -- Regards, Tim Shen commit

Do not produce empty try-finally statements

2014-01-19 Thread Jan Hubicka
Hi, when compiling: struct A { virtual int foo (void) {return 42;} }; int test (void) { struct A a, *b=a; return b-foo(); } We wind up the following useless EH: int test() () { int D.2235; int (*__vtbl_ptr_type) () * D.2236; int (*__vtbl_ptr_type) () D.2237; struct A a;

RE: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8

2014-01-19 Thread Joey Ye
Ping -Original Message- From: Joey Ye [mailto:joey...@arm.com] Sent: Thursday, January 16, 2014 16:28 To: gcc-patches@gcc.gnu.org Subject: [PATCH][4.8] Backport strict-volatile-bitfields fixes to 4.8 4.8 has a number of strict-volatile-bitfields issues that can be fixed by

Update MAINTAINERS (Re: Remove spam in GCC mailing list)

2014-01-19 Thread Gerald Pfeifer
On Sun, 22 Dec 2013, Tae Wong wrote: It will be better if you have enabled the Launchpad account seotaewong40. This account has been suspended for renaming an answer title multiple times. The GCC team has nothing to do with Launchpad. There's also an ordering error on the GCC maintainer

RE: [GOMP4][PATCH] SIMD-enabled functions (formerly Elemental functions) for C++

2014-01-19 Thread Iyer, Balaji V
Hi Jakub, I have fixed all the issues you have mentioned below. I am also attaching a fixed patch along with ChangeLog entries. I have also rebased the patch to the trunk revision r206786. Is this Ok to install? Thanks, Balaji V. Iyer. -Original Message- From: