Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Marc Glisse
On Wed, 5 Sep 2012, Gabriel Dos Reis wrote: On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: Let's say we have two for loops like this: int my_func (int x, int y); For (ii = 0; ii 1; ii++) X[ii] = my_func (Y[ii], Z[ii]); I assume X, Y and Z are

Re: [PATCH] Fix sel-sched ICE with asm goto (PR rtl-optimization/54455)

2012-09-06 Thread Andrey Belevantsev
Hello, On 05.09.2012 19:38, Jakub Jelinek wrote: Hi! As discussed in the PR, sel-sched doesn't handle correctly tidying of empty blocks if fallthru predecessor ends with asm goto that has some labels on the empty block in addition to the fallthru edge. cfgrtl.c can handle that, so this patch

Re: [PATCH] Fix PR 54494, removal of volatile store in strlen optimization because of the inlininer

2012-09-06 Thread Jakub Jelinek
On Wed, Sep 05, 2012 at 09:10:53PM -0700, Andrew Pinski wrote: The inlininer likes to recreate some MEM_REF, it copies most of the bits (TREE_THIS_NOTRAP, TREE_THIS_VOLATILE, etc.) but forgets about TREE_SIDE_EFFECTS. This causes the strlen optimization to think the memory store does not

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Dominique Dhumieres
Oleg, Bootstrap fails at revision 190996 on powerpc-apple-darwin9 with: g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -fno-common

Re: [PATCH] Fix PR 54494, removal of volatile store in strlen optimization because of the inlininer

2012-09-06 Thread Andrew Pinski
On Thu, Sep 6, 2012 at 12:07 AM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Sep 05, 2012 at 09:10:53PM -0700, Andrew Pinski wrote: The inlininer likes to recreate some MEM_REF, it copies most of the bits (TREE_THIS_NOTRAP, TREE_THIS_VOLATILE, etc.) but forgets about TREE_SIDE_EFFECTS.

Re: [PATCH] Fix PR 54494, removal of volatile store in strlen optimization because of the inlininer

2012-09-06 Thread Jakub Jelinek
On Thu, Sep 06, 2012 at 12:50:44AM -0700, Andrew Pinski wrote: Patch preapproved, but you've attached a different patch. Sorry about that. Here is the correct one. Also is this ok for the 4.7 branch? Yes, thanks. --- tree-inline.c (revision 191004) +++ tree-inline.c (working

Re: Ping: [PATCH GCC/ARM] Fix problem that hardreg_cprop opportunities are missed on thumb1

2012-09-06 Thread Richard Earnshaw
On 06/09/12 06:41, Bin Cheng wrote: Hi Richard, Thanks very much for comments. Ping? Hi Ramana, could you help me review this patch? Hi Eric, Richard, could you help me review the change in regcprop.c? Subtraction of zero isn't canonical rtl though. Passes after peephole2 would be

RE: Ping: [PATCH GCC/ARM] Fix problem that hardreg_cprop opportunities are missed on thumb1

2012-09-06 Thread Bin Cheng
Yes, it may be feasible to rewrite the instruction in machine reorg pass, rather than peephole2. But that need bigger change in ARM back end. Hi Ramana, Richard, what's your opinion on this? Thanks very much. I side with Richard on this one. The mid-end should only have to deal

Re: [Patch ARM] implement bswap16

2012-09-06 Thread Richard Earnshaw
On 05/09/12 17:01, Christophe Lyon wrote: Hi, This patch implements __builtin_bswap16() on ARM (v6 and above) using revsh with a signed input and rev16 with an unsigned input. It is pretty much equal to the patch posted some time ago

Re: Change double_int calls to new interface.

2012-09-06 Thread Richard Guenther
On Wed, 5 Sep 2012, Lawrence Crowl wrote: On 9/5/12, Richard Guenther rguent...@suse.de wrote: On Tue, 4 Sep 2012, Lawrence Crowl wrote: Modify gcc/*.[hc] double_int call sites to use the new interface. This change entailed adding a few new methods to double_int. Other changes will

[PATCH] Make -fno-tree-pta work

2012-09-06 Thread Richard Guenther
This fixes -fno-tree-pta - noticed it doesn't work when trying to compile a testcase where PTA uses too much memory. Bootstrapped tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-09-06 Richard Guenther rguent...@suse.de * passes.c (execute_function_todo): Call

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Oleg Endo
On Thu, 2012-09-06 at 09:49 +0200, Dominique Dhumieres wrote: Oleg, Bootstrap fails at revision 190996 on powerpc-apple-darwin9 with: g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long

Re: [patch] Random cleanups

2012-09-06 Thread Richard Guenther
On Wed, Sep 5, 2012 at 10:34 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, Just some cleanups I did while working on something bigger. OK for trunk? Ok. Thanks, Richard. Ciao! Steven * graphite.c (print_global_statistics): Use EDGE_COUNT instead of

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Dominique Dhumieres
Could you please commit this (I can't at the moment)? Sorry, I don't have write access. Dominique

Re: [patch] Fix bitmap_last_set_bit

2012-09-06 Thread Richard Guenther
On Wed, Sep 5, 2012 at 10:40 PM, Steven Bosscher stevenb@gmail.com wrote: Hi, bitmap.c:bitmap_last_set_bit() is not used by any code in the current GCC trunk, but I'm using it and I noticed it returns an incorrect result. This patch rewrites most of the function to return the correct

Re: Ping: [PATCH GCC/ARM] Fix problem that hardreg_cprop opportunities are missed on thumb1

2012-09-06 Thread Richard Earnshaw
On 06/09/12 09:33, Bin Cheng wrote: Yes, it may be feasible to rewrite the instruction in machine reorg pass, rather than peephole2. But that need bigger change in ARM back end. Hi Ramana, Richard, what's your opinion on this? Thanks very much. I side with Richard on this one. The

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Richard Guenther
On Thu, Sep 6, 2012 at 8:06 AM, Marc Glisse marc.gli...@inria.fr wrote: On Wed, 5 Sep 2012, Gabriel Dos Reis wrote: On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: Let's say we have two for loops like this: int my_func (int x, int y); For (ii = 0; ii 1;

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Oleg Endo
On 6 Sep 2012, at 11:18, domi...@lps.ens.fr (Dominique Dhumieres) wrote: Could you please commit this (I can't at the moment)? Sorry, I don't have write access. OK, then I will commit it later today. Cheers, Oleg

Re: [PATCH] Combine location with block using block_locations

2012-09-06 Thread Richard Guenther
On Wed, Aug 22, 2012 at 1:54 AM, Dehao Chen de...@google.com wrote: On Tue, Aug 21, 2012 at 6:25 AM, Richard Guenther richard.guent...@gmail.com wrote: On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen de...@google.com wrote: ping Conceptually I like the change. Can a libcpp maintainer please

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Gabriel Dos Reis
On Thu, Sep 6, 2012 at 1:06 AM, Marc Glisse marc.gli...@inria.fr wrote: On Wed, 5 Sep 2012, Gabriel Dos Reis wrote: On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: Let's say we have two for loops like this: int my_func (int x, int y); For (ii = 0; ii 1;

RE: [PATCH] Enable bbro for -Os

2012-09-06 Thread Zhenqiang Chen
Thank you for the comments. -Original Message- From: Eric Botcazou [mailto:ebotca...@adacore.com] Sent: Wednesday, September 05, 2012 9:39 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Enable bbro for -Os Basic block reordering is disabled for -Os from

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Marc Glisse
On Thu, 6 Sep 2012, Marc Glisse wrote: AFAIU, my_func is defined in a separate library and because of the attribute on the definition, it will actually export overloads: int myfunc(int,int); v2si myfunc(v2si,v2si); v4si myfunc(v4si,v4si); etc (where does it stop? seems problematic if the

[PATCH] Remove unused MOVE_NONTEMPORAL

2012-09-06 Thread Richard Guenther
This removes the MOVE_NONTEMPORAL tree flag which is nowhere set. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-09-06 Richard Guenther rguent...@suse.de * tree.h (MOVE_NONTEMPORAL): Remove. * tree-pretty-print.c (dump_generic_node): Remove

Re: [PATCH] Fix SCCVN aggregate copy code some more

2012-09-06 Thread H.J. Lu
On Thu, Aug 11, 2011 at 8:31 AM, Richard Guenther rguent...@suse.de wrote: When adjusting the C++ FE code generation for base type copies I noticed the XFAILs on g++.dg/tree-ssa/pr41186.C and investigated things a bit. We can fix the testcase easily iff the frontend emits the MEM_REF

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Georg-Johann Lay
Oleg Endo schrieb: On Wed, 2012-09-05 at 14:39 -0400, DJ Delorie wrote: I don't feel the m32c change needs my specific ack, it's a harmless change that goes with the ack for the feature itself. However, I will note that m32c does have different costs for addresses in different address spaces,

[PATCH] Fix PR54498

2012-09-06 Thread Richard Guenther
The following fixes PR54498 - we may not treat already visited parts of the CFG as non-aliasing after re-writing the expression we lookup. Just abort the walk in this case. Bootstrapped and tested on x86_64-unknown-linux-gnu on the 4.7 branch, trunk testing in progress. Richard. 2012-09-06

[PATCH] rs6000 TARGET_ADDRSS_COST

2012-09-06 Thread David Edelsohn
rs6000_debug_address_cost() needs to call TARGET_ADDRESS_COST with the new arguments. Committed. - David * config/rs6000/rs6000.c (rs6000_xcoff_asm_named_section): Add TLS section. * config/rs6000/rs6000.c (rs6000_debug_address_cost): Add new arguments to

[AArch64] allow 16 bytes constants in constant pool.

2012-09-06 Thread Marcus Shawcroft
Relax the logic that prevents TFmode constants being addressed in the constant pool. 2012-09-06 Marcus Shawcroft marcus.shawcr...@arm.com * config/aarch64/aarch64.c (aarch64_classify_address): Allow 16 byte modes in constant pool. diff --git a/gcc/config/aarch64/aarch64.c

[AArch64] Implement section anchors

2012-09-06 Thread James Greenhalgh
Hi, This patch implements section anchors for the AArch64 port. OK for aarch64-branch? Regards, James Greenhalgh -- 2012-09-06 James Greenhalgh james.greenha...@arm.com Richard Earnshaw rearn...@arm.com * common/config/aarch64/aarch64-common.c

[AArch64] support long double exceptions and rounding mode

2012-09-06 Thread Marcus Shawcroft
Enable the raising of exception in long double soft float and support for rounding mode. 2012-09-06 Marcus Shawcroft marcus.shawcr...@arm.com * config/aarch64/sfp-machine.h (FP_EX_INVALID, FP_EX_DIVZERO) (FP_EX_OVERFLOW, FP_EX_UNDERFLOW, FP_EX_INEXACT)

Re: [AArch64] Implement section anchors

2012-09-06 Thread Richard Earnshaw
On 06/09/12 15:31, James Greenhalgh wrote: Hi, This patch implements section anchors for the AArch64 port. OK for aarch64-branch? Regards, James Greenhalgh -- 2012-09-06 James Greenhalgh james.greenha...@arm.com Richard Earnshaw rearn...@arm.com *

RE: [PATCH, AArch64] Allow symbol+offset even if not being used for memory access

2012-09-06 Thread Ian Bolton
On 2012-08-31 07:49, Ian Bolton wrote: +(define_split + [(set (match_operand:DI 0 register_operand =r) + (const:DI (plus:DI (match_operand:DI 1 aarch64_valid_symref S) + (match_operand:DI 2 const_int_operand i] + + [(set (match_dup 0) (high:DI

Re: [PATCH, AArch64] Allow symbol+offset even if not being used for memory access

2012-09-06 Thread Richard Henderson
On 09/06/2012 08:06 AM, Ian Bolton wrote: If I don't use my split pattern, I could alter combine to remove the requirement that parent is a MEM. What do you think? I merely question the calling out of CONST as special. Either you've got some pattern that handles SYMBOL_REF the same way, or

C++ PATCH for c++/54341, c++/54253 (constexpr and virtual functions)

2012-09-06 Thread Jason Merrill
Vtables were causing several different problems for constexpr: 1) Value-initializing a nearly-empty class (that has a vptr but no data) meant two initializers for a single base. Fixed by not bothering to zero out a type with no data before calling its constructor. 2) A primary base is

PR testsuite/54184: rewriting failing data race test

2012-09-06 Thread Aldy Hernandez
The current test is failing on some architectures because the underlying gimple has changed. I believe the best way to test the speculative store data race is with the simulate-thread/ harness. We already have a speculative store test in the harness, so this will be a nice addition. I have

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Richard Henderson
On 09/06/2012 02:37 AM, Richard Guenther wrote: In all this seems unrelated to CILK+ work (even if you make use of this from within CILK+). While true, we also asked him to split up the work. And this piece, done correctly, seems useful even if the rest of cilk is ignored. r~

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Iyer, Balaji V
Hello Joseph, Thanks for reviewing my patch. Please see my responses below: -Original Message- From: Joseph Myers [mailto:jos...@codesourcery.com] Sent: Wednesday, September 05, 2012 8:07 PM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org; Aldy Hernandez (al...@redhat.com); Jeff

Re: [Patch ARM] implement bswap16

2012-09-06 Thread Christophe Lyon
On 6 September 2012 10:48, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 17:01, Christophe Lyon wrote: +(define_insn *arm_revsh + [(set (match_operand:SI 0 s_register_operand =r) + (sign_extend:SI (bswap:HI (match_operand:HI 1 s_register_operand r] + TARGET_32BIT

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Richard Henderson
On 09/05/2012 03:09 PM, Iyer, Balaji V wrote: If we annotate *both* the function declaration and the function with the following attribute, the compiler will create a vector and scalar version of the function. __attribute__((vector)) my_func (int x, int y); __attribute__((vector))

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Iyer, Balaji V
Hello Marc, Please see my response below. Thanks for looking at my patch! Sincerely, Balaji V. Iyer. -Original Message- From: Marc Glisse [mailto:marc.gli...@inria.fr] Sent: Thursday, September 06, 2012 2:06 AM To: Gabriel Dos Reis Cc: Iyer, Balaji V; gcc-patches@gcc.gnu.org;

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Gabriel Dos Reis
On Thu, Sep 6, 2012 at 10:51 AM, Richard Henderson r...@redhat.com wrote: On 09/06/2012 02:37 AM, Richard Guenther wrote: In all this seems unrelated to CILK+ work (even if you make use of this from within CILK+). While true, we also asked him to split up the work. And this piece, done

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Joseph S. Myers
On Thu, 6 Sep 2012, Iyer, Balaji V wrote: Ok, I was mistaken there. I thought we had to add a changelog entry for every function and not every file. I will fix it in the updated patch I send soon. For functions in existing files you do need to mention each function - but not for new files.

Re: [C++ Patch] PR 18747

2012-09-06 Thread Paolo Carlini
Hi, On 09/06/2012 02:03 AM, Jason Merrill wrote: but note that for: template class T struct A { int select() { return 0; } }; we have parser-num_template_parameter_lists == 1 and num_templates == 0. Thus it seems that the case 'num_templates + 1' isn't (just) about member templates...

Re: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Gabriel Dos Reis
On Thu, Sep 6, 2012 at 11:11 AM, Iyer, Balaji V balaji.v.i...@intel.com wrote: On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: Let's say we have two for loops like this: int my_func (int x, int y); For (ii = 0; ii 1; ii++) X[ii] = my_func

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Iyer, Balaji V
Sorry, I didn't see this message. Please see my responses below: -Original Message- From: Marc Glisse [mailto:marc.gli...@inria.fr] Sent: Thursday, September 06, 2012 7:04 AM To: gcc-patches@gcc.gnu.org Cc: Gabriel Dos Reis; Iyer, Balaji V; Aldy Hernandez (al...@redhat.com); Jeff Law;

Re: [Patch ARM] implement bswap16

2012-09-06 Thread Richard Earnshaw
On 06/09/12 17:07, Christophe Lyon wrote: On 6 September 2012 10:48, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 17:01, Christophe Lyon wrote: +(define_insn *arm_revsh + [(set (match_operand:SI 0 s_register_operand =r) + (sign_extend:SI (bswap:HI (match_operand:HI 1

Re: [PATCH, ARM] Cleanup: Replace GET_CODE comparisons with predicates

2012-09-06 Thread Richard Earnshaw
On 06/09/12 12:29, Kyrylo Tkachov wrote: 2012-09-06 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/arm/arm.c: Use CONST_INT_P, CONST_DOUBLE_P, REG_P, MEM_P, LABEL_P, JUMP_P, CALL_P, NOTE_P, BARRIER_P consistently. * config/arm/arm.h: Use REG_P, MEM_P

RE: [PATCH, AArch64] Allow symbol+offset even if not being used for memory access

2012-09-06 Thread Ian Bolton
From: Richard Henderson [mailto:r...@redhat.com] On 09/06/2012 08:06 AM, Ian Bolton wrote: If I don't use my split pattern, I could alter combine to remove the requirement that parent is a MEM. What do you think? I merely question the calling out of CONST as special. Either you've

Re: [C++ Patch] PR 18747

2012-09-06 Thread Jason Merrill
On 09/06/2012 12:18 PM, Paolo Carlini wrote: On 09/06/2012 02:03 AM, Jason Merrill wrote: but note that for: template class T struct A { int select() { return 0; } }; we have parser-num_template_parameter_lists == 1 and num_templates == 0. Thus it seems that the case 'num_templates + 1'

Bump minimum gmp version to 4.2.3

2012-09-06 Thread Diego Novillo
Doug was trying to build with gmp 4.2.1 and ran into this error: gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared This bug was fixed in gmp 4.2.3, so I've bumped the minimum acceptable version in configure.ac Tested on x86_64 with gmp 4.2.3. Committed to trunk. Diego.

[PATCH] OpenBSD/hppa support

2012-09-06 Thread Mark Kettenis
Most bits are stolen from Linux, but there are a few subtle differences since our assembler is configured to be slightly more HP-UX-ish. libgcc/: 2012-09-06 Mark Kettenis kette...@openbsd.org * config.host (hppa-*-openbsd*): New target. * config/pa/t-openbsd: New file. gcc/:

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Iyer, Balaji V
-Original Message- From: Joseph Myers [mailto:jos...@codesourcery.com] Sent: Thursday, September 06, 2012 12:18 PM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org; Aldy Hernandez (al...@redhat.com); Jeff Law; r...@redhat.com Subject: RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of

[PATCH] Fix PR bootstrap/54419

2012-09-06 Thread Jack Howarth
The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by having configure check for assembler support of the new rdrnd opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported. Tested on x86_64-apple-darwin12 against the assembler from Xcode 4.4.1. Okay for

Re: [PATCH, libstdc++] Improve slightly __cxa_guard_acquire

2012-09-06 Thread Benjamin De Kosnik
On Thu, 30 Aug 2012 12:48:34 +0200 Thiago Macieira thiago.macie...@intel.com wrote: Hello The attached patch is a simple improvement to make a thread that failed to set the waiting bit to exit the function earlier, if it detects that another thread has successfully finished initialising.

Re: [PATCH] Fix PR bootstrap/54419

2012-09-06 Thread Ulrich Drepper
On Thu, Sep 6, 2012 at 2:40 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: Okay for gcc trunk? One typo: * configure.ac: Test for rdrnd support in assembler. It's rdrand. I wouldn't be pedantic if the opcode wouldn't have changed from rdrnd to rdrand at some point and using the old

RE: [PATCH] Merging Cilk Plus into Trunk (Patch 1 of approximately 22)

2012-09-06 Thread Joseph S. Myers
On Thu, 6 Sep 2012, Iyer, Balaji V wrote: Concepts such as pentium_4 are architecture-specific and have no place in front-end files. This whole mapping from one sort of string to another belongs within the back end. Please excuse me if I am beating this horse to death. I am asking this

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-06 Thread Oleg Endo
On Thu, 2012-09-06 at 14:41 +0200, Georg-Johann Lay wrote: Oleg Endo schrieb: On Wed, 2012-09-05 at 14:39 -0400, DJ Delorie wrote: I don't feel the m32c change needs my specific ack, it's a harmless change that goes with the ack for the feature itself. However, I will note that m32c

Re: Bump minimum gmp version to 4.2.3

2012-09-06 Thread NightStrike
On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo dnovi...@google.com wrote: Doug was trying to build with gmp 4.2.1 and ran into this error: gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared This bug was fixed in gmp 4.2.3, so I've bumped the minimum acceptable version in

Re: Bump minimum gmp version to 4.2.3

2012-09-06 Thread NightStrike
On Thu, Sep 6, 2012 at 3:13 PM, NightStrike nightstr...@gmail.com wrote: On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo dnovi...@google.com wrote: Doug was trying to build with gmp 4.2.1 and ran into this error: gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared This bug

[PATCH][revised] Fix PR bootstrap/54419

2012-09-06 Thread Jack Howarth
The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by having configure check for assembler support of the new rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported. Tested on x86_64-apple-darwin12 against the assembler from Xcode 4.4.1. Okay for

Re: [PATCH][revised] Fix PR bootstrap/54419

2012-09-06 Thread Jakub Jelinek
On Thu, Sep 06, 2012 at 03:22:10PM -0400, Jack Howarth wrote: The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by having configure check for assembler support of the new rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported. Tested on

Re: [PATCH][revised] Fix PR bootstrap/54419

2012-09-06 Thread Jakub Jelinek
On Thu, Sep 06, 2012 at 03:22:10PM -0400, Jack Howarth wrote: The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by having configure check for assembler support of the new rdrand opcode and defining _GLIBCXX_X86_RDRAND in config.h if supported. Tested on

Re: Bump minimum gmp version to 4.2.3

2012-09-06 Thread Diego Novillo
On 2012-09-06 15:13 , NightStrike wrote: On Thu, Sep 6, 2012 at 2:13 PM, Diego Novillo dnovi...@google.com wrote: Doug was trying to build with gmp 4.2.1 and ran into this error: gmp-4.2.1/include/gmp.h:515:12: error: 'std::FILE' has not been declared This bug was fixed in gmp 4.2.3, so I've

Re: Bump minimum gmp version to 4.2.3

2012-09-06 Thread Diego Novillo
On 2012-09-06 15:16 , NightStrike wrote: Actually, on closer inspection, other places list the minimum version as 4.3.2, including: http://gcc.gnu.org/install/prerequisites.html as well as the two locations that I mention above. Perhaps configure should do likewise. It already does

[PATCH][revisedx2] Fix PR bootstrap/54419

2012-09-06 Thread Jack Howarth
The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by adding a define to acinclude.m4, GLIBCXX_CHECK_X86_RDRAND, that checks for assembler support of the new rdrnd opcode, using this new define in configure.ac and also defining _GLIBCXX_X86_RDRAND in config.h if

Re: [PATCH, libstdc++] Improve slightly __cxa_guard_acquire

2012-09-06 Thread Thiago Macieira
On quinta-feira, 6 de setembro de 2012 13.33.11, Benjamin De Kosnik wrote: Here's the patch as applied to trunk in rev. 191042. I'll apply it to 4.7 this weekend as long as nobody yelps. Thanks. The change to ACQUIRE is also a bugfix. -- Thiago Macieira - thiago.macieira (AT) intel.com

Re: [PATCH, libstdc++] Improve slightly __cxa_guard_acquire

2012-09-06 Thread Jakub Jelinek
On Thu, Sep 06, 2012 at 01:33:11PM -0700, Benjamin De Kosnik wrote: Here's the patch as applied to trunk in rev. 191042. I'll apply it to 4.7 this weekend as long as nobody yelps. 2012-09-06 Thiago Macieira thiago.macie...@intel.com PR libstdc++/54172 * libsupc++/guard.cc

[PATCH, i386]: Remove .code64 from HLE configure test

2012-09-06 Thread Uros Bizjak
Hello! We can use 32bit address without .code64 directive; this test is valid for 32bit and 64bit targets. 2012-09-06 Uros Bizjak ubiz...@gmail.com * configure.ac (hle prefixes): Remove .code64. * configure: Regenerated. Tested on x86_64-pc-linux-gnu, committed to mainline

[google] Added new dump flag -pmu to display pmu data in pass summaries (issue6489092)

2012-09-06 Thread Chris Manghane
This patch adds a new dump flag that dumps PMU profile information using the -pmu dump option. This patch should be applied to google/main. Tested with crosstools. 2012-09-06 Chris Manghane cm...@google.com * gcc/doc/invoke.texi: Modified pmu-profile-use option. *

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread Andrew Pinski
On Thu, Sep 6, 2012 at 2:55 PM, James Lemke jwle...@codesourcery.com wrote: Attached are the patches for this gcc port. On a recent checkout (r191027) I have run the DejaGNU suite with no new failures for binutils, gas, ld, gcc, g++, gfortran. A bootstrap is in progress. Could you explain

Re: [C++ Patch] PR 18747

2012-09-06 Thread Paolo Carlini
Hi, On 09/06/2012 07:47 PM, Jason Merrill wrote: On 09/06/2012 12:18 PM, Paolo Carlini wrote: On 09/06/2012 02:03 AM, Jason Merrill wrote: but note that for: template class T struct A { int select() { return 0; } }; we have parser-num_template_parameter_lists == 1 and num_templates ==

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread Maciej W. Rozycki
On Thu, 6 Sep 2012, Andrew Pinski wrote: Could you explain why you are changing system.h ? Also seems like TARGET_VLE_ISEL should not be needed TARGET_ISEL is always set for VLE targets. You mean this: + POWERPC_E200_MASK = MASK_VLE | MASK_ISEL | MASK_MULTIPLE ? Well, this just marks

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread Andrew Pinski
On Thu, Sep 6, 2012 at 3:39 PM, Maciej W. Rozycki ma...@codesourcery.com wrote: On Thu, 6 Sep 2012, Andrew Pinski wrote: Could you explain why you are changing system.h ? Also seems like TARGET_VLE_ISEL should not be needed TARGET_ISEL is always set for VLE targets. You mean this: +

Re: [PATCH][revisedx2] Fix PR bootstrap/54419

2012-09-06 Thread Paolo Carlini
Hi, On 09/06/2012 10:32 PM, howa...@frodo.msbb.uc.edu wrote: The attached patch eliminates the bootstrap failures in libstdc++-v3 of PR 54419 by adding a define to acinclude.m4, GLIBCXX_CHECK_X86_RDRAND, that checks for assembler support of the new rdrnd opcode, using this new define in

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread Maciej W. Rozycki
On Thu, 6 Sep 2012, Andrew Pinski wrote: You mean this: + POWERPC_E200_MASK = MASK_VLE | MASK_ISEL | MASK_MULTIPLE ? Well, this just marks that the e200 processor supports ISEL regardless of the mode selected (standard vs VLE). Then with -mvle ISEL is supposed to be enabled

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread Joseph S. Myers
On Thu, 6 Sep 2012, James Lemke wrote: Attached are the patches for this gcc port. On a recent checkout (r191027) I have run the DejaGNU suite with no new failures for binutils, gas, ld, gcc, g++, gfortran. A bootstrap is in progress. The -t* options added duplicate -mcpu= options; the

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread James Lemke
On 09/06/2012 06:09 PM, Andrew Pinski wrote: Could you explain why you are changing system.h ? That was a convenience to me at one point. It should have been deleted from the patch set. -- Jim Lemke Mentor Graphics / CodeSourcery Orillia Ontario, +1-613-963-1073

[gccgo branch] Add cherry picked bug fixes

2012-09-06 Thread Ian Lance Taylor
I have committed this patch to the gccgo branch to add some cherry picked bug fixes to the Go library. This is for internal release purposes. The mainline and 4.7 tree remain at the Go 1.0.2 release. I expect that most of these patches will be in Go 1.0.3. The complete list of cherry-picked

[patch trivial] Fix comment in dwarf2.def

2012-09-06 Thread Cary Coutant
I've committed this trivial comment patch to src/include/dwarf2.def (and will shortly do the same in gcc/include). -cary 2012-09-06 Cary Coutant ccout...@google.com include/ * dwarf2.def: Edit comment. diff --git a/include/dwarf2.def b/include/dwarf2.def index 3c3dfcc..7fe2df1

Re: [PATCH] PowerPC VLE port

2012-09-06 Thread James Lemke
On 09/06/2012 07:07 PM, Joseph S. Myers wrote: The -t* options added duplicate -mcpu= options; the only existing precedent appears to be arm-vxworks and I don't think the options are appropriate for generic PowerPC target files (not specific to an OS port such as VxWorks with its own special

Re: [google] Added new dump flag -pmu to display pmu data in pass summaries (issue6489092)

2012-09-06 Thread Teresa Johnson
On Thu, Sep 6, 2012 at 2:49 PM, Chris Manghane cm...@google.com wrote: This patch adds a new dump flag that dumps PMU profile information using the -pmu dump option. This patch should be applied to google/main. Tested with crosstools. 2012-09-06 Chris Manghane cm...@google.com

Re: [google] Added new dump flag -pmu to display pmu data in pass summaries (issue6489092)

2012-09-06 Thread Teresa Johnson
On Thu, Sep 6, 2012 at 5:34 PM, Chris Manghane cm...@google.com wrote: On Thu, Sep 6, 2012 at 5:08 PM, Teresa Johnson tejohn...@google.com wrote: On Thu, Sep 6, 2012 at 2:49 PM, Chris Manghane cm...@google.com wrote: This patch adds a new dump flag that dumps PMU profile information using

Re: [patch, mips] New mips triplet for multilib linux builds

2012-09-06 Thread Steve Ellcey
On Thu, 2012-09-06 at 06:47 +0100, Richard Sandiford wrote: Is this an 'if-then-else' usage? Yeah, but I typoed, sorry. It should be: %{mips32r2|mips64r2:-msynci;:-mno-synci} Richard OK, I got that working now. I am still having some issues though. My original patch was setup to

Re: [PATCH] Remove RS6000_CALL_GLUE

2012-09-06 Thread David Edelsohn
On Sun, Aug 26, 2012 at 7:03 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: On all supported targets, plain nop is the correct thing to use (cror 31,31,31 was for POWER). Tested on powerpc64-linux --enable-languages=c,c++,fortran; no regressions. Okay for mainline? Segher

[google] Added new dump flag -pmu to display pmu data in pass summaries (issue6489092)

2012-09-06 Thread Chris Manghane
Fixed spacing and condensed repeated code into helper. This patch should be applied to google/main. Tested with crosstools. 2012-09-06 Chris Manghane cm...@google.com * gcc/doc/invoke.texi: Modified pmu-profile-use option. * gcc/tree-dump.c: Added new dump flag. *

[PATCH] Add -fmem-report-wpa

2012-09-06 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com For parallel LTO builds setting -fmem-report does not work very well because all the LTRANS phases dump it in parallel and typically interleave it to unreadability. Since usually the memory bottleneck is WPA add a flag to only dump the memory report for

[PATCH] Fix part of PR gcov-profile/54487 (issue6501100)

2012-09-06 Thread Teresa Johnson
This fixes part of the issue described in PR gcov-profile/54487 where there were warnings about mismatches due to slight differences in the merged histograms in different object files. This can happen due to the truncating integer division in the merge routine, which could result in slightly