Re: Fix compute_reloc_for_constant

2014-01-19 Thread Bernhard Reutner-Fischer
On 19 January 2014 03:12:56 Jan Hubicka 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 decisions based on

[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 __bui

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

[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/ * g

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 auxilia

Re: [PATCH] Fix crossing jumps in functions with forced_labels (PR rtl-optimization/57763)

2014-01-19 Thread Eric Botcazou
> 2014-01-18 Jakub Jelinek > > PR rtl-optimization/57763 > * bb-reorder.c (fix_crossing_unconditional_branches): Set JUMP_LABEL > on the new indirect jump_insn and increment LABEL_NUSES (label). OK, thanks. -- Eric Botcazou

Re: [PATCH][buildrobot] vax: Fix unused variable warning

2014-01-19 Thread Gerald Pfeifer
Jan-Benedict Glaw wrote > * config/vax/vax.h (FUNCTION_ARG_REGNO_P): Vix unused variable >warning. Fix? ;-) And if it's been broken for a while, just go ahead and commit the fix. Gerald

Re: Commit: MSP430: Add -mcpu= option

2014-01-19 Thread Gerald Pfeifer
Nick Clifton wrote: > I am committing the attached patch to add a -mcpu= command line option > to the MSP430 backend. This allows the selection of the ISA to be > used, and it operates independently of the -mmcu= command line option. Mind adding an entry to the release notes? Happy to help if y

Re: Fix compute_reloc_for_constant

2014-01-19 Thread Richard Biener
On Sun, Jan 19, 2014 at 3:12 AM, Jan Hubicka 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 dec

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 wrote: > On Sat, Jan 18, 2014 at 9:15 PM, H.J. Lu 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

[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 Description

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

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 wrote: > On Sun, Jan 19, 2014 at 3:20 PM, H.J. Lu 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) .

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

[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 -ffat-lto-o

[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? Th

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 unmatch

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

2014-01-19 Thread Richard Sandiford
Jakub Jelinek 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 writes: >> > expand_movstr is work fine when we don't define movstr pattern or >> > always

[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 a

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 v

[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 $$s

[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 * 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 tar

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, > D

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 fo

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 * scripts/create_testsuite_files: Add testsuite/experimental in the list of folders to introspect for tests. * include/experimental/string_view (basic_string

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 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 5e36fe9d0d2dbbb2141016d

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; stru

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: Jak

Re: Do not produce empty try-finally statements

2014-01-19 Thread Jakub Jelinek
On Sun, Jan 19, 2014 at 11:46:09PM +0100, Jan Hubicka wrote: > Bootstrapped/regtested x86_64-linux, OK? This looks very wrong. By ignoring the clobbers in the cleanups you ensure clobbers are hardly ever emitted, but the TRY/FINALLY is the way how to make sure the clobbers appear in all the place