Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Eric Botcazou
> This is not the global constructor/destructor issue with names > generated by collect2. But ELF or SVR4 is able to provide a unique > name without resorting to random numbers. Not clear if the name needs be unique at all if you have the right features, see name-lookup.c:get_anonymous_namespace

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Arnaud Charlet
> * Similarly to IRIX 6, there was a mismatch in ada/init.c for the type > of the signal handler installed with sigaction. g++ also doesn't like > arithmetic on void * ;-) I'm a bit puzzled as to why the C compiler isn't complaining, and what error message the C++ compiler is generating, can

Re: [Patch, Fortran] PR49791 - Fix legacy namelist support

2011-07-20 Thread Steve Kargl
On Thu, Jul 21, 2011 at 07:59:36AM +0200, Tobias Burnus wrote: > Steve Kargl wrote: > >The patch looks incomplete. Where is gfc_warning("Extension: expanding > >namelist with nonstandard array syntax")? > > There exists *no* gfc_warning in libgfortran. And as it is not a > compiled file but fil

Re: [Patch, Fortran] PR49791 - Fix legacy namelist support

2011-07-20 Thread Tobias Burnus
Steve Kargl wrote: The patch looks incomplete. Where is gfc_warning("Extension: expanding namelist with nonstandard array syntax")? There exists *no* gfc_warning in libgfortran. And as it is not a compiled file but file read by the compiled program, there is no possibility to diagnose this a

[v3] constexpr array additions

2011-07-20 Thread Benjamin Kosnik
Also pointed out by Daniel. It's kind of hokey to work around -fno-exceptions like this, but seems harmless. In this particular case, deferring the throw to a function like __throw_out_of_range makes the ::at member an invalid constant expression. Yay! std::array with compile time operator []!

PATCH [9/n] X32: PR target/49798: Zero-extend symbol address to 64bit if needed

2011-07-20 Thread H.J. Lu
The testcase is gcc.c-torture/compile/pr45728.c. This patch zero-extends symbol address to 64bit if needd. OK for trunk? Thanks. H.J. 2011-07-20 H.J. Lu PR target/49798 * config/i386/i386.c (ix86_asm_integer): New. (TARGET_ASM_INTEGER): Likewise. diff --git a/

PR 45819 - possible fix?

2011-07-20 Thread DJ Delorie
In this PR, a cast to a volatile type is lost during forwprop1, resulting in the wrong access semantics being used for a memory-mapped peripheral register. Checking for loss of volatile in this patch solves the problem, but I don't know if this is the right place to put such a test - it could go

[v3] constexpr time_point additions

2011-07-20 Thread Benjamin Kosnik
This updates to N3291, and adds constexpr to time_point nonmember arithmetic operations, as pointed out to me in private email by Daniel Krugler. Indeed, they can become constant expressions: thanks! These additions are on top of N3229. tested x86/linux -benjamin2011-07-20 Benjamin Kosnik

Re: Remove NetWare support

2011-07-20 Thread Benjamin Kosnik
> diff --git a/libstdc++-v3/crossconfig.m4 b/libstdc++-v3/crossconfig.m4 > --- a/libstdc++-v3/crossconfig.m4 > +++ b/libstdc++-v3/crossconfig.m4 > @@ -173,14 +173,6 @@ case "${host}" in >AC_DEFINE(HAVE_ISNANL) > fi > ;; > - *-netware) > -SECTION_FLAGS='-ffunction-sections -f

[pph] Add filter to prevent streaming out builtin identifiers (issue4802047)

2011-07-20 Thread Gabriel Charette
Changed to use new cpp_is_builtin inline function. Gab diff --git a/libcpp/ChangeLog.pph b/libcpp/ChangeLog.pph index de21994..36369a5 100644 --- a/libcpp/ChangeLog.pph +++ b/libcpp/ChangeLog.pph @@ -1,3 +1,9 @@ +2011-07-20 Gabriel Charette + + * include/cpplib.h (cpp_is_builtin): New. +

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread David Edelsohn
On Wed, Jul 20, 2011 at 4:53 PM, Mike Stump wrote: > On Jul 20, 2011, at 1:25 PM, Ian Lance Taylor wrote: >> Interesting.  I don't know why this doesn't happen on GNU/Linux. > > It doesn't happen as the symbols in question are local on linux.  collect2 > runs nm on public symbols looking for symb

Re: [PATCH 4/6] Shrink-wrapping

2011-07-20 Thread Bernd Schmidt
On 07/07/11 16:34, Richard Sandiford wrote: > Is JUMP_LABEL ever null after this change? (In fully-complete rtl > sequences, I mean.) It looked like some of the null checks in the > patch might not be necessary any more. It turns out that computed jumps can have a NULL JUMP_LABEL, and so can JUM

Re: [Patch, Fortran] PR49791 - Fix legacy namelist support

2011-07-20 Thread Steve Kargl
On Thu, Jul 21, 2011 at 12:30:53AM +0200, Tobias Burnus wrote: > > Build and regtested on x86-64-linux. OK for the trunk? > The patch looks incomplete. Where is gfc_warning("Extension: expanding namelist with nonstandard array syntax")? -- Steve

Re: [PATCH] Dump basic_block flags.

2011-07-20 Thread Richard Henderson
On 07/20/2011 03:23 PM, Steven Bosscher wrote: > On Thu, Jul 21, 2011 at 12:16 AM, Richard Henderson wrote: >> + static const char * const bits[] = { >> +"new", "reachable", "irr_loop", "superblock", "disable_sched", >> +"hot_partition", "cold_partition", "duplicated", >> +

Re: [RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 2:49 PM, Uros Bizjak wrote: > On Wed, Jul 20, 2011 at 9:46 PM, Uros Bizjak wrote: > >> I have looked at example in rs6000.c, the only target that uses >> SUBREG_PROMOTED_UNSIGNED_P. Looking at other sources, S_P_U_P is used >> in conjunction with SUBREG_PROMOTED_VAR. It lo

Re: introduce --param max-vartrack-expr-depth

2011-07-20 Thread Michael Eager
On 07/20/2011 01:48 PM, Jakub Jelinek wrote: On Wed, Jul 20, 2011 at 01:28:46PM -0700, Michael Eager wrote: On 07/20/2011 01:23 PM, Jakub Jelinek wrote: On Wed, Jul 20, 2011 at 01:07:40PM -0700, Michael Eager wrote: I've run into a problem with this change when building microblaze-xilinx-elf.

[Patch, Fortran] PR49791 - Fix legacy namelist support

2011-07-20 Thread Tobias Burnus
This patch partially undoes the change done in PR 46010. For that patch, first (comment 7) + || !dtp->u.p.ionml->touched) was added - and after not solving the problem, the complete change was (comment 9 and committed version) + || !dtp->u.p.ionml->touched +

Re: CFT: Move unwinder to toplevel libgcc

2011-07-20 Thread Steve Ellcey
On Tue, 2011-07-19 at 12:47 +0200, Rainer Orth wrote: > unfortunately, I don't even have an idea what this error is supposed to > mean. Seems to be an error ultimately due to bfd/elfxx-ia64.c > (elfNN_ia64_size_dynamic_sections) failing. > > To debug this, I'd start by comparing the link lines f

Re: [PATCH] Dump basic_block flags.

2011-07-20 Thread Steven Bosscher
On Thu, Jul 21, 2011 at 12:16 AM, Richard Henderson wrote: > + static const char * const bits[] = { > + "new", "reachable", "irr_loop", "superblock", "disable_sched", > + "hot_partition", "cold_partition", "duplicated", > + "non_local_goto_target", "rtl", "forwarder",

[PATCH] Dump basic_block flags.

2011-07-20 Thread Richard Henderson
Previously we had no indication of which partition a block has been placed in. Re-use some existing code, rather than duplicate it in print_rtl_with_bb. Bootstrapped on x86_64-linux. r~ * cfg.c (dump_bb_info): Dump basic_block->flags. * cfgrtl.c (print_rtl_with_bb): Use dump_bb_

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ian Lance Taylor
"Ralf Wildenhues" writes: > Hello, > > * Ian Lance Taylor wrote on Wed, Jul 20, 2011 at 08:06:51PM CEST: >> Rainer Orth writes: >> >> > diff --git a/gcc/configure.ac b/gcc/configure.ac >> > --- a/gcc/configure.ac >> > +++ b/gcc/configure.ac >> > @@ -1041,7 +1041,16 @@ case "${host}" in >> > es

Re: [C++-0x] User defined literals.

2011-07-20 Thread Jason Merrill
On 07/20/2011 05:32 PM, Ed Smith-Rowland wrote: Now I have a real question: Since a shadowed suffix *cannot* act like the user wants no matter what, should I error instead of warn? It doesn't seem any more erroneous than any other suffix without a leading underscore, even though it isn't very

Re: [RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread Uros Bizjak
On Wed, Jul 20, 2011 at 9:46 PM, Uros Bizjak wrote: > I have looked at example in rs6000.c, the only target that uses > SUBREG_PROMOTED_UNSIGNED_P. Looking at other sources, S_P_U_P is used > in conjunction with SUBREG_PROMOTED_VAR. It looks to me that using the > combination should be OK to dete

[PATCH, i386]: Allow subregs of multi-word values in addresses

2011-07-20 Thread Uros Bizjak
On Wed, Jul 20, 2011 at 9:46 PM, Uros Bizjak wrote: >> Note that SUBREG_PROMOTED_UNSIGNED_P wasn't designed for paradoxical subregs, >> but for regular subregs (typically of word-sized objects).  You should check >> that the ones created for x32 (because of POINTERS_EXTEND_UNSIGNED I guess) >> ar

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Ian Lance Taylor
Mike Stump writes: >> Presumably the fix will be to use -frandom-seed. > > But, the random seem was to ensure that things that should not collide, > don't. If you use 0, then things that should not collide, eventually will > and your world with then end. In the present code base, it is perfec

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 2:24 PM, Richard Henderson wrote: > On 07/20/2011 02:00 PM, H.J. Lu wrote: >>        operands[0] = expand_simple_binop (Pmode, code, op0, op1, NULL_RTX, 0, >>                                        OPTAB_DIRECT); >>      } >> +  else if (TARGET_X32) >> +    operands[0] = co

Re: [C++-0x] User defined literals.

2011-07-20 Thread Ed Smith-Rowland
On 07/12/2011 04:56 PM, Jason Merrill wrote: A few more notes: + if (DECL_NAMESPACE_SCOPE_P (decl)) + { + if (!check_literal_operator_args(decl, + &long_long_unsigned_p, &long_double_p)) + { + error ("%qD has illegal argument list", decl); +

[Patch, Fortran] Coarray: Pass token for coarray dummies

2011-07-20 Thread Tobias Burnus
With -fcoarray=lib, coarrays are identified to the library by a "token". Thus, asking for a RHS expression like "caf(2)[4]" means that one requests from the library to read 4-bytes from image 4 from the coarray identified by "token" starting from an offset of 4-bytes, assuming that the array is

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread Richard Henderson
On 07/20/2011 02:00 PM, H.J. Lu wrote: >operands[0] = expand_simple_binop (Pmode, code, op0, op1, NULL_RTX, 0, >OPTAB_DIRECT); > } > + else if (TARGET_X32) > +operands[0] = convert_memory_address (Pmode, operands[0]); > }) > > (define_ins

Re: [pph] Add filter to prevent streaming out builtin identifiers (issue4802047)

2011-07-20 Thread dnovillo
On 2011/07/20 20:54:31, Gabriel Charette wrote: Should I instead add a new macro IS_BUILTIN or something like that to encompass that logic? Sure. But make it a static inline function, please. OK with that change. Diego. http://codereview.appspot.com/4802047/

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Toon Moene
On 07/20/2011 04:34 PM, Toon Moene wrote: So I changed my "lto" bootstrap script to do the following: language=fortran if [ $RANDOM -lt 16384 ] then language=ada fi ... ../gcc/configure \ ... --enable-languages=c++,$language \ Still have to see if this will fit in the 2:20 hour gap between two

Re: [PATCH] split mips/gnu-user.h from mips/linux.h

2011-07-20 Thread Richard Sandiford
Robert Millan writes: > 2011/7/19 Richard Sandiford : >> needs to deal with linux64.h as well.  I also kept the copyright notice >> the same on linux.h (even though it's only a one-liner now). >> >> Applied with those changes, thanks. > > It's probably harmless, but with this change if --enable-ta

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 1:09 PM, H.J. Lu wrote: > On Wed, Jul 20, 2011 at 10:31 AM, Richard Henderson wrote: >> On 07/20/2011 10:25 AM, H.J. Lu wrote: >>> I am testing this patch.  OK for trunk if it works? >> >> Yep. >> > > It doesn't work.  Looking into it. > We have #define CASE_VECTOR_MODE

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ralf Wildenhues
Hello, * Ian Lance Taylor wrote on Wed, Jul 20, 2011 at 08:06:51PM CEST: > Rainer Orth writes: > > > diff --git a/gcc/configure.ac b/gcc/configure.ac > > --- a/gcc/configure.ac > > +++ b/gcc/configure.ac > > @@ -1041,7 +1041,16 @@ case "${host}" in > > esac > > AC_FUNC_FORK > > > > +# FIXME:

[pph] Add filter to prevent streaming out builtin identifiers (issue4802047)

2011-07-20 Thread Gabriel Charette
Added a filter to prevent streaming out builtin identifiers. This filter is copied from symtab.c:497 where the same filter already existed. Should I instead add a new macro IS_BUILTIN or something like that to encompass that logic? This doesn't fix any current pph test. Tested with bootstrap b

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Mike Stump
On Jul 20, 2011, at 1:25 PM, Ian Lance Taylor wrote: > Interesting. I don't know why this doesn't happen on GNU/Linux. It doesn't happen as the symbols in question are local on linux. collect2 runs nm on public symbols looking for symbols of a particular form, it then builds two lists, one for

Re: introduce --param max-vartrack-expr-depth

2011-07-20 Thread Jakub Jelinek
On Wed, Jul 20, 2011 at 01:28:46PM -0700, Michael Eager wrote: > On 07/20/2011 01:23 PM, Jakub Jelinek wrote: > >On Wed, Jul 20, 2011 at 01:07:40PM -0700, Michael Eager wrote: > >>I've run into a problem with this change when building > >>microblaze-xilinx-elf. > >> > >>When compiling _divdi3.o, c

Re: introduce --param max-vartrack-expr-depth

2011-07-20 Thread Michael Eager
On 07/20/2011 01:23 PM, Jakub Jelinek wrote: On Wed, Jul 20, 2011 at 01:07:40PM -0700, Michael Eager wrote: I've run into a problem with this change when building microblaze-xilinx-elf. When compiling _divdi3.o, cselib_expand_value_rtx_1 returns a huge rtx tree for variable _r1 when max_depth i

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Ian Lance Taylor
David Edelsohn writes: > I now can get through the build of the compiler, but stage2 and stage3 > libstdc++ and libsupc++ files have many comparison failures due to > tree.c:get_file_function_name() introducing explicit randomness to > produce different symbols for anonymous namespaces: Interest

Re: introduce --param max-vartrack-expr-depth

2011-07-20 Thread Jakub Jelinek
On Wed, Jul 20, 2011 at 01:07:40PM -0700, Michael Eager wrote: > I've run into a problem with this change when building microblaze-xilinx-elf. > > When compiling _divdi3.o, cselib_expand_value_rtx_1 returns a huge rtx > tree for variable _r1 when max_depth is greater than 17. If -g is > specified

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 10:31 AM, Richard Henderson wrote: > On 07/20/2011 10:25 AM, H.J. Lu wrote: >> I am testing this patch.  OK for trunk if it works? > > Yep. > It doesn't work. Looking into it. -- H.J.

Re: introduce --param max-vartrack-expr-depth

2011-07-20 Thread Michael Eager
On 05/31/2011 09:13 AM, Alexandre Oliva wrote: On May 30, 2011, Bernd Schmidt wrote: On 05/30/2011 12:35 PM, Alexandre Oliva wrote: One of my patches for PR 48866 regressed guality/asm-1.c on x86_64-linux-gnu because what used to be a single complex debug value expression became a chain of de

Re: [RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread Uros Bizjak
On Wed, Jul 20, 2011 at 8:04 PM, Eric Botcazou wrote: >> Another improvement is introduction of SUBREG_PROMOTED_UNSIGNED_P >> predicate in the same area to handle paradoxical SUBREGs. When SUBREG >> satisfies this predicate, the compiler guarantees, that excess bits >> are zero (see the documentat

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread David Edelsohn
I now can get through the build of the compiler, but stage2 and stage3 libstdc++ and libsupc++ files have many comparison failures due to tree.c:get_file_function_name() introducing explicit randomness to produce different symbols for anonymous namespaces: /* Otherwise, the name must be uniq

Re: RFA: PATCHes to accept C99 designated initializer syntax in C++

2011-07-20 Thread Jakub Jelinek
On Wed, Jul 20, 2011 at 02:13:55PM -0400, Jason Merrill wrote: > The first patch adjusts the C++ front end's current support for the > old GNU designated initializer syntax to support the C99 syntax as > well. Thanks. > The second patch adjusts recog.h/genoutput.c to use a new macro > HAVE_DESIGN

Re: bb-reorder maintenance [3/n]

2011-07-20 Thread Ulrich Weigand
Richard Henderson wrote: > On 07/20/2011 11:24 AM, Ulrich Weigand wrote: > >> >BB_HEAD (new_bb) = new_label; > > new_bb is now always NULL at this point, and the assignment crashes. > > > > I guess that assignment should now be just deleted as well, since > > this is done by cr

Re: bb-reorder maintenance [3/n]

2011-07-20 Thread Richard Henderson
On 07/20/2011 11:24 AM, Ulrich Weigand wrote: >> > BB_HEAD (new_bb) = new_label; > new_bb is now always NULL at this point, and the assignment crashes. > > I guess that assignment should now be just deleted as well, since > this is done by create_basic_block later? Doh. Yes, that's

Re: [patch] Fix PR tree-optimization/49771

2011-07-20 Thread Ulrich Weigand
Ira Rosen wrote: >PR tree-optimization/49771 >* gcc.dg/vect/pr49771.c: New test. This test fails (with wrong code) on spu-elf ... > +int > +foo (void) > +{ > + int j; > + int i; > + for (i = 0; i < 1000; i++) > +for (j = 0; j < 1000; j++) > + a[j] = a[i] + 1; > + return a[0]

Re: bb-reorder maintenance [3/n]

2011-07-20 Thread Ulrich Weigand
Richard Henderson wrote: > The create_basic_block function will do the right thing > with ordering the block label and the block note. No > need for us to work hard at that within bb-reorder. This causes: FAIL: gcc.dg/tree-prof/bb-reorg.c compilation, -fprofile-use -D_PROFILE_USE (internal co

[v3] Use noexcept in

2011-07-20 Thread Paolo Carlini
Hi, tested x86_64-linux multilib, committed. Thanks, Paolo. 2011-07-20 Paolo Carlini * include/std/system_error: Use noexcept. * src/system_error.cc: Likewise. * testsuite/19_diagnostics/error_condition/modifiers/39881.cc: Adjust. * tests

RFA: PATCHes to accept C99 designated initializer syntax in C++

2011-07-20 Thread Jason Merrill
The first patch adjusts the C++ front end's current support for the old GNU designated initializer syntax to support the C99 syntax as well. The second patch adjusts recog.h/genoutput.c to use a new macro HAVE_DESIGNATED_UNION_INITIALIZERS instead of HAVE_DESIGNATED_INITIALIZERS because with t

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ian Lance Taylor
Rainer Orth writes: > diff --git a/gcc/ada/init.c b/gcc/ada/init.c > --- a/gcc/ada/init.c > +++ b/gcc/ada/init.c > @@ -1031,7 +1031,7 @@ __gnat_install_handler (void) > exceptions. Make sure that the handler isn't interrupted by another > signal that might cause a scheduling event!

Re: [PATCH, PR 49094] Refrain from creating misaligned accesses in SRA

2011-07-20 Thread Ulrich Weigand
Richard Guenther wrote: > On Tue, Jul 19, 2011 at 8:20 PM, Ulrich Weigand wrote: > > The problem is that in this expression > > disappear = VIEW_CONVERT_EXPR(x_8); > > the rhs is considered unaligned and blocks the SRA transformation. > > > > The check you added for SSA_NAMEs doesn't hit, becaus

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ian Lance Taylor
Rainer Orth writes: > diff --git a/libcpp/system.h b/libcpp/system.h > --- a/libcpp/system.h > +++ b/libcpp/system.h > @@ -1,6 +1,6 @@ > /* Get common system includes and various definitions and declarations based > on autoconf macros. > - Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003,

Re: [Dwarf-Discuss] Vendor extensions in .debug_macinfo

2011-07-20 Thread Michael Eager
On 07/20/2011 11:00 AM, Jakub Jelinek wrote: On Wed, Jul 20, 2011 at 10:16:10AM -0700, Michael Eager wrote: It took me a few days to review the current DWARF macinfo specification and review this proposal. The existing macro data format is clearly in need of revision. I don't think that there

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ian Lance Taylor
Rainer Orth writes: > diff --git a/gcc/configure.ac b/gcc/configure.ac > --- a/gcc/configure.ac > +++ b/gcc/configure.ac > @@ -1041,7 +1041,16 @@ case "${host}" in > esac > AC_FUNC_FORK > > +# FIXME: g++ on Solaris 10+ defines _XOPEN_SOURCE=600, which exposes a > +# different iconv() prototyp

Re: Named address spaces broken (Re: [2/5] Add a set_mem_attrs function)

2011-07-20 Thread Ulrich Weigand
Richard Guenther wrote: > Ok. Checked in, thanks. > Do we have any chance of adding generic testcases for named-address-space > support that would show these issues? Not really; named address space are fundamentally target-specific. It certainly would help if a more common target provided some

Re: [RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread Eric Botcazou
> Another improvement is introduction of SUBREG_PROMOTED_UNSIGNED_P > predicate in the same area to handle paradoxical SUBREGs. When SUBREG > satisfies this predicate, the compiler guarantees, that excess bits > are zero (see the documentation). This is exaclty what we want in > registers that form

[PATCH] Make rs6000 port bootstrap using G++ as 2nd/3rd stage compilers

2011-07-20 Thread Michael Meissner
I tried building the powerpc64-linux compiler today, and it would not bootstrap, since evidently stages 2 and 3 are built with G++ instead of C, and G++ is more strict about const pointers. This patch allows the compiler to bootstrap. Is it ok to install? 2011-07-20 Michael Meissner

Re: [Dwarf-Discuss] Vendor extensions in .debug_macinfo

2011-07-20 Thread Jakub Jelinek
On Wed, Jul 20, 2011 at 10:16:10AM -0700, Michael Eager wrote: > It took me a few days to review the current DWARF macinfo specification > and review this proposal. > > The existing macro data format is clearly in need of revision. I don't > think that there is any way to modify this format whic

Re: [testsuite]: Some test case skips.

2011-07-20 Thread Mike Stump
On Jul 14, 2011, at 8:28 AM, Georg-Johann Lay wrote: > This patchlet fixes two test cases: > > * gcc.dg/pr32912-2.c: TImode is overkill for AVR. > * gcc.dg/pr44674.c: Test if -fprofile-generate is available. > > Ok to commit? Ok. [ Sorry, I missed this one. ]

Re: [build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Ian Lance Taylor
Rainer Orth writes: > diff --git a/gcc/config/sol2-c.c b/gcc/config/sol2-c.c > --- a/gcc/config/sol2-c.c > +++ b/gcc/config/sol2-c.c > @@ -68,7 +68,7 @@ static const format_char_info cmn_err_ch >{ NULL, 0, STD_C89, NOLENGTHS, NULL, NULL, NULL } > }; > > -const format_kind_info solaris_for

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread Richard Henderson
On 07/20/2011 10:25 AM, H.J. Lu wrote: > I am testing this patch. OK for trunk if it works? Yep. r~

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 9:42 AM, Richard Henderson wrote: >> -       ix86_print_operand (file, x, 0); >> +       /* Always use 64bit register for indirect branch.  */ >> +       if (REG_P (x) && TARGET_64BIT) >> +         print_reg (x, 'q', file); >> +       else >> +         ix86_print_operand (f

Re: [Dwarf-Discuss] Vendor extensions in .debug_macinfo

2011-07-20 Thread Michael Eager
On 07/13/2011 11:20 AM, Jakub Jelinek wrote: Hi! The current .debug_macinfo format isn't very much extensible and is not really designed to allow reducing of duplicate debug info, so when compilers want to produce maco information, size of .debug_macinfo is prohibitive. We are discussing some e

Re: [Patch,AVR]: Fix PR36467, PR49687 (better widening mul)

2011-07-20 Thread Richard Henderson
On 07/20/2011 04:12 AM, Georg-Johann Lay wrote: > Johann > > PR target/36467 > PR target/49687 > * config/avr/avr.md (mulhi3): Use register_or_s9_operand for > operand2 and expand appropriately if there is a CONST_INT in > operand2. > (usmulqihi3): New insn. >

Re: [patch tree-optimization]: [2 of 3]: Boolify compares & more

2011-07-20 Thread Kai Tietz
2011/7/20 Richard Guenther : > On Wed, Jul 20, 2011 at 3:31 PM, Kai Tietz wrote: >> 2011/7/20 Richard Guenther : >>> On Wed, Jul 20, 2011 at 3:05 PM, Kai Tietz wrote: Hello, this is the revised version of the partial pre-approved patch for preserving type-casts from/to b

Re: [RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread Richard Henderson
On 07/20/2011 09:41 AM, Uros Bizjak wrote: > * config/i386/i386.c (ix86_decompose_address): Also allow promoted > paradoxical subregs in base and PLUS chains. Allow only paradoxical > subregs and subregs of DImode hard registers in subregs of index. > (ix86_legitimate_addre

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread Richard Henderson
> - ix86_print_operand (file, x, 0); > + /* Always use 64bit register for indirect branch. */ > + if (REG_P (x) && TARGET_64BIT) > + print_reg (x, 'q', file); > + else > + ix86_print_operand (file, x, 0); > return; This bit is because of ... > +(de

[RFC PATCH, i386]: Allow SUBREG_PROMOTED_UNSIGNED_P subregs in address

2011-07-20 Thread Uros Bizjak
Hello! Attached RFC patch buids on recent x86 improvements in ix86_legitimate_address_p/ix86_decompose_address functions. With this patch, we allow SUBREGs with register_no_elim_operand registers in base, index and PLUS chains. As the consequence, we can relax the condition that rejects SUBREGs, w

[build, ada] Allow Solaris bootstrap with C++ (PR bootstrap/49794)

2011-07-20 Thread Rainer Orth
As partially described in the PR, Solaris 10 bootstrap is broken with C++: * In both libcpp and gcc, ICONV_CONST is misdetected since the configure test is run with gcc, but the actual compilation performed with g++. Unlike the former, the latter defines _XOPEN_SOURCE=600 (for the benefit of

Re: [Patch,AVR]: Fix rtx_costs of SYMBOL_REF

2011-07-20 Thread Richard Henderson
On 07/20/2011 07:16 AM, Georg-Johann Lay wrote: > case CONST_INT: > case CONST_DOUBLE: > +case SYMBOL_REF: >/* Immediate constants are as cheap as registers. */ >*total = 0; >return true; > @@ -5348,7 +5349,6 @@ avr_rtx_costs (rtx x, int codearg, int o >

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 8:43 AM, Uros Bizjak wrote: > On Wed, Jul 20, 2011 at 4:09 PM, H.J. Lu wrote: > Hello! > >>> +(define_insn "*tablejump_1_x32" >>> +  [(set (pc) (match_operand:SI 0 "register_operand" "r")) >>> +   (use (label_ref (match_operand 1 "" "")))] >>> +  "TARGET_X32" >>> +  "jmp\t

Re: Allow IRIX Ada bootstrap with C++

2011-07-20 Thread Arnaud Charlet
> With this patch, bootstrap continued. Ok for mainline if it passes? OK, thanks. Arno

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Ian Lance Taylor
David Edelsohn writes: > AIX needs libsupc++ for libstdc++ static linking. > > * Makefile.tpl (POSTSTAGE1_CONFIGURE_FLAGS): Add libsupc++ to > link directories. > * Makefile.in: Rebuild. > > Index: Makefile.tpl > ===

Allow IRIX Ada bootstrap with C++

2011-07-20 Thread Rainer Orth
A bootstrap on IRIX 6.5 only saw a single issue with C++ so far: ada/init.c failed to compile: /vol/gcc/src/hg/trunk/local/gcc/ada/init.c: In function 'void __gnat_install_handler()': /vol/gcc/src/hg/trunk/local/gcc/ada/init.c:862:20: error: invalid conversion from 'void (*)(int, int, sigcontext

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Thomas Schwinge
Hi! On Wed, 20 Jul 2011 11:41:37 -0400, David Edelsohn wrote: > AIX needs libsupc++ for libstdc++ static linking. > > * Makefile.tpl (POSTSTAGE1_CONFIGURE_FLAGS): Add libsupc++ to > link directories. > * Makefile.in: Rebuild. > > Index: Makefile.tpl > ===

Re: [v3] add

2011-07-20 Thread Paolo Carlini
On 07/20/2011 05:34 PM, Ed Smith-Rowland wrote: Here is a trivial addition to include scoped_allocator in the precompiled headers. It bootstrapped and regtested on x86_64 linux. Applied, thanks. Paolo.

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread Uros Bizjak
On Wed, Jul 20, 2011 at 4:09 PM, H.J. Lu wrote: Hello! >> +(define_insn "*tablejump_1_x32" >> +  [(set (pc) (match_operand:SI 0 "register_operand" "r")) >> +   (use (label_ref (match_operand 1 "" "")))] >> +  "TARGET_X32" >> +  "jmp\t%A0" >> +  [(set_attr "type" "ibr") >> +   (set_attr "length_im

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread David Edelsohn
AIX needs libsupc++ for libstdc++ static linking. * Makefile.tpl (POSTSTAGE1_CONFIGURE_FLAGS): Add libsupc++ to link directories. * Makefile.in: Rebuild. Index: Makefile.tpl === --- Makefile.tpl(revision 17648

Re: [v3] add

2011-07-20 Thread Ed Smith-Rowland
On 07/18/2011 12:18 PM, Jonathan Wakely wrote: On 9 July 2011 14:46, Jonathan Wakely wrote: This adds the new header. Currently only std::vector has the necessary support for C++0x allocators to be usable with std::scoped_allocator_adaptor. * include/Makefile.am: Add new header.

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Jonathan Wakely
Feel the burn^Wchurn! On 20/07/2011, Ian Lance Taylor wrote: > Diego Novillo writes: > >> On Wed, Jul 20, 2011 at 08:41, Richard Guenther >> wrote: >> >>> Which is good as it increases testing coverage.  We probably would have >>> missed this bug completely if you wouldn't have notice it. >>

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Ian Lance Taylor
Toon Moene writes: > On 07/19/2011 08:33 PM, Ian Lance Taylor wrote: > >>> 2011-07-15 Ian Lance Taylor >>> >>> * configure.ac: Add --enable-build-poststage1-with-cxx. If set, >>> make C++ a boot_language. Set and substitute >>> POSTSTAGE1_CONFIGURE_FLAGS. >>> * Makefile.tpl (PO

Re: [patch] Reduce over-promotion of vector operations

2011-07-20 Thread Ira Rosen
On 20 July 2011 17:28, Richard Guenther wrote: > On Tue, Jul 19, 2011 at 10:57 AM, Richard Guenther > wrote: >> On Tue, Jul 19, 2011 at 8:44 AM, Ira Rosen wrote: >>> Hi, >>> >>> This patch tries to reduce over-promotion of vector operations that >>> could be done with narrower elements, e.g., fo

RE: [Patch,AVR]: Fix rtx_costs of SYMBOL_REF

2011-07-20 Thread Weddington, Eric
> -Original Message- > From: Georg-Johann Lay [mailto:a...@gjlay.de] > Sent: Wednesday, July 20, 2011 8:16 AM > To: gcc-patches@gcc.gnu.org > Cc: Anatoly Sokolov; Denis Chertykov; Weddington, Eric; Richard Henderson > Subject: [Patch,AVR]: Fix rtx_costs of SYMBOL_REF > > This is a mini-p

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Toon Moene
On 07/19/2011 08:33 PM, Ian Lance Taylor wrote: 2011-07-15 Ian Lance Taylor * configure.ac: Add --enable-build-poststage1-with-cxx. If set, make C++ a boot_language. Set and substitute POSTSTAGE1_CONFIGURE_FLAGS. * Makefile.tpl (POSTSTAGE1_CONFIGURE_FLAGS): Ne

Re: [patch] Reduce over-promotion of vector operations

2011-07-20 Thread Richard Guenther
On Tue, Jul 19, 2011 at 10:57 AM, Richard Guenther wrote: > On Tue, Jul 19, 2011 at 8:44 AM, Ira Rosen wrote: >> Hi, >> >> This patch tries to reduce over-promotion of vector operations that >> could be done with narrower elements, e.g., for >> >> char a; >> int b, c; >> short d; >> >> b = (int)

C++ PATCH for c++/42603 and c++/6709 (DR 743/950, allow decltype as base-specifier and nested-name-specifier)

2011-07-20 Thread Jason Merrill
DRs 743 and 950 allow decltype to be used as the scope in a nested-name-specifier and as a base-specifier. This patch implements that functionality. In order to deal with the ambiguity when we first encounter "decltype" as to whether it will be a nested-name-specifier or its own simple-type-s

[Patch,AVR]: Fix rtx_costs of SYMBOL_REF

2011-07-20 Thread Georg-Johann Lay
This is a mini-patch to fix rtx_costs for SYMBOL_REF. The cost of a SYMBOL_REF is the same as a CONST_INT because it can be used the same way. I observed small performance regression in my code and found that the reason is too high cost for SYMBOL_REF. char x[10]; char getx2 (unsigned char i, u

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 6:41 AM, Uros Bizjak wrote: > On Wed, Jul 20, 2011 at 3:18 PM, H.J. Lu wrote: >> On Wed, Jul 20, 2011 at 1:19 AM, Uros Bizjak wrote: >>> On Wed, Jul 20, 2011 at 9:53 AM, Uros Bizjak wrote: >>> since Pmode is used in non-PIC tablejump, we have to put 64bit value

Re: Make regcprop check HARD_REGNO_MODE_OK

2011-07-20 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/20/11 07:47, Richard Sandiford wrote: > This patch makes regcprop check HARD_REGNO_MODE_OK before creating a > hard register in a different mode. It fixes a segfault in a build of > bionic on ARM. > > The missing check usually doesn't cause pro

Re: PATCH RFA: Build stages 2 and 3 with C++

2011-07-20 Thread Ian Lance Taylor
Diego Novillo writes: > On Wed, Jul 20, 2011 at 08:41, Richard Guenther > wrote: > >> Which is good as it increases testing coverage.  We probably would have >> missed this bug completely if you wouldn't have notice it. > > Agreed. The pain we feel due to this is similar to the pain one feels >

Re: [patch tree-optimization]: [2 of 3]: Boolify compares & more

2011-07-20 Thread Richard Guenther
On Wed, Jul 20, 2011 at 3:31 PM, Kai Tietz wrote: > 2011/7/20 Richard Guenther : >> On Wed, Jul 20, 2011 at 3:05 PM, Kai Tietz wrote: >>> Hello, >>> >>> this is the revised version of the partial pre-approved patch for preserving >>> type-casts from/to boolean-types.  It fixes additionally the re

Make regcprop check HARD_REGNO_MODE_OK

2011-07-20 Thread Richard Sandiford
This patch makes regcprop check HARD_REGNO_MODE_OK before creating a hard register in a different mode. It fixes a segfault in a build of bionic on ARM. The missing check usually doesn't cause problems. The define_insn constraints are likely to reject invalid registers for "real" insns, while de

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread Uros Bizjak
On Wed, Jul 20, 2011 at 3:18 PM, H.J. Lu wrote: > On Wed, Jul 20, 2011 at 1:19 AM, Uros Bizjak wrote: >> On Wed, Jul 20, 2011 at 9:53 AM, Uros Bizjak wrote: >> >>> since Pmode is used in non-PIC tablejump, we have to put 64bit value for >>> labels with 0 upper 32bits in tablejump for x32

[Committed] S/390: Fix return type of s390_class_max_nregs

2011-07-20 Thread Andreas Krebbel
Hi, this fixes a bug which has been revealed by compiling the back-end with g++ now. Committed to mainline and 4.6. Bye, -Andreas- 2011-07-20 Andreas Krebbel * config/s390/s390.c (s390_class_max_nregs): Fix return type. * config/s390/s390-protos.h (s390_class_max_nregs): L

Re: [patch tree-optimization]: [2 of 3]: Boolify compares & more

2011-07-20 Thread Kai Tietz
2011/7/20 Richard Guenther : > On Wed, Jul 20, 2011 at 3:05 PM, Kai Tietz wrote: >> Hello, >> >> this is the revised version of the partial pre-approved patch for preserving >> type-casts from/to boolean-types.  It fixes additionally the regression in >> tree-ssa/builtin-expect-5.c testcase, which

Ping #1: [testsuite]: Some test case skips.

2011-07-20 Thread Georg-Johann Lay
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01166.html Georg-Johann Lay wrote: > This patchlet fixes two test cases: > > * gcc.dg/pr32912-2.c: TImode is overkill for AVR. > * gcc.dg/pr44674.c: Test if -fprofile-generate is available. > > Ok to commit? > > Johann > > testsuite/ > * gcc.d

Re: PATCH [7/n] X32: Handle address output and calls patterns

2011-07-20 Thread H.J. Lu
On Wed, Jul 20, 2011 at 1:19 AM, Uros Bizjak wrote: > On Wed, Jul 20, 2011 at 9:53 AM, Uros Bizjak wrote: > >> since Pmode is used in non-PIC tablejump, we have to put 64bit value for >> labels with 0 upper 32bits in tablejump for x32. > > The mode is completely controled by CASE_

Re: [patch tree-optimization]: [2 of 3]: Boolify compares & more

2011-07-20 Thread Richard Guenther
On Wed, Jul 20, 2011 at 3:05 PM, Kai Tietz wrote: > Hello, > > this is the revised version of the partial pre-approved patch for preserving > type-casts from/to boolean-types.  It fixes additionally the regression in > tree-ssa/builtin-expect-5.c testcase, which was caused by fold_builtin_expect.

  1   2   >