mark expected failures for ppc64 (issue6932046)

2012-12-10 Thread Jing Yu
Add powerpc64-grtev3-linux-gnu.xfail to mark expected failures for powerpc64 toolchain. For google/gcc_4-7 branch. Tested: ./buildit --build_type=symlinks --keep_work_dir --run_tests gcc-4.7.x-grtev3-powerpc64 2012-12-10 Jing Yu * contrib/testsuite-management/powerpc64-grtev3-linu

Re: application/xml mime-type in recent libstdc++ doc changes

2012-12-10 Thread Matthias Klose
Am 10.12.2012 18:52, schrieb Benjamin De Kosnik: > >>> libstdc++-v3/doc/xsl/customization.xsl.in is marked as >>> >>> svn:mime-type = application/xml >>> >>> at least on the 4.7 branch, having some unexpected outcome for svn >>> diff. If this was unintended, could you change the svn:mime-type >>

Re: [GOOGLE] disable streaming out TREE_BLOCK to cure lto-bootstrap

2012-12-10 Thread Dehao Chen
On Mon, Dec 10, 2012 at 9:30 PM, Andrew Pinski wrote: > > On Mon, Dec 10, 2012 at 9:23 PM, Dehao Chen wrote: > > Hi, > > > > The location_block patch has failed lto-bootstrap. This is fixed by a > > bunch of fixes in trunk. But we would rather not spend too much effort > > to back-port those fixe

Re: [GOOGLE] disable streaming out TREE_BLOCK to cure lto-bootstrap

2012-12-10 Thread Andrew Pinski
On Mon, Dec 10, 2012 at 9:23 PM, Dehao Chen wrote: > Hi, > > The location_block patch has failed lto-bootstrap. This is fixed by a > bunch of fixes in trunk. But we would rather not spend too much effort > to back-port those fixes. So for now, we would disable streaming out > TREE_BLOCK as a worka

Re: [GOOGLE] disable streaming out TREE_BLOCK to cure lto-bootstrap

2012-12-10 Thread Xinliang David Li
ok. David On Mon, Dec 10, 2012 at 9:23 PM, Dehao Chen wrote: > Hi, > > The location_block patch has failed lto-bootstrap. This is fixed by a > bunch of fixes in trunk. But we would rather not spend too much effort > to back-port those fixes. So for now, we would disable streaming out > TREE_BLOC

[GOOGLE] disable streaming out TREE_BLOCK to cure lto-bootstrap

2012-12-10 Thread Dehao Chen
Hi, The location_block patch has failed lto-bootstrap. This is fixed by a bunch of fixes in trunk. But we would rather not spend too much effort to back-port those fixes. So for now, we would disable streaming out TREE_BLOCK as a workaround to solve to lto-bootstrap. Bootstrapped and passed cross

[cxx-conversion] Fix hash_table build problems with checking enabled.

2012-12-10 Thread Lawrence Crowl
Fix some hash_table build errors when configured with --enable-checking=yes. tree-browser.c * Remove stale declaration of removed TB_parent_eq. * Fix template parameter for base class to match value_type. gimple.h * Use gimplify_hasher::hash rather than gimple_tree_hash in the asserti

Re: [Patch, Fortran, F03] PR52909: Procedure pointers not private to modules

2012-12-10 Thread Janus Weil
Hi Tobias, > I think the patch below is okay, especially in light of patch PR49110/51055 > which also contains a slight ABI breakage. thanks, committed as r194375. > Can you mention the potential ABI > issue in the release notes (gfortran wiki). Will do ... Cheers, Janus > On 09.04.2012 18

Re: [PATCH] Bugfix: Additional parameter for canonicalize comparison

2012-12-10 Thread Kaz Kojima
Andreas Krebbel wrote: > Index: gcc/config/sh/sh.c > === > *** gcc/config/sh/sh.c.orig > --- gcc/config/sh/sh.c [snop] > ! static void > sh_canonicalize_comparison (enum rtx_code& cmp, rtx& op0, rtx& op1, > !

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-10 Thread Richard Henderson
On 12/10/2012 08:50 AM, Steve Ellcey wrote: > 2012-12-07 Steve Ellcey > > PR target/54061 > rtl.h (IGNORED_DWARF_REGNUM): New. > * dwarf2out.c (reg_loc_descriptor): Check for IGNORED_DWARF_REGNUM. > (mem_loc_descriptor): Ditto. > * config/mips/mips.h (ALL_COP_REG_F

Re: [PATCH] PR c++/53609 - Wrong argument deduction for pack expansion in argument pack

2012-12-10 Thread Jason Merrill
On 12/08/2012 05:12 PM, Dodji Seketeli wrote: + else if (arg_from_pack_level_to_prune || has_empty_arg) +{ + /* ... we just return a pack expansion which pattern is PATTERN +into which ARGS has been substituted. */ + *instantiation_yields_no_list_p = true; +} Though

Re: [asan] Handle noreturn calls with __asan_handle_no_return ()

2012-12-10 Thread Jakub Jelinek
On Mon, Dec 10, 2012 at 10:44:49PM +0100, Dodji Seketeli wrote: > Jakub Jelinek writes: > > > +++ gcc/asan.c 2012-12-05 15:30:56.069890542 +0100 > > @@ -1031,7 +1031,7 @@ instrument_builtin_call (gimple_stmt_ite > > { > >gimple call = gsi_stmt (*iter); > > > > - gcc_assert (is_gimple

Re: [asan] Handle noreturn calls with __asan_handle_no_return ()

2012-12-10 Thread Dodji Seketeli
Jakub Jelinek writes: > +++ gcc/asan.c2012-12-05 15:30:56.069890542 +0100 > @@ -1031,7 +1031,7 @@ instrument_builtin_call (gimple_stmt_ite > { >gimple call = gsi_stmt (*iter); > > - gcc_assert (is_gimple_builtin_call (call)); > + gcc_checking_assert (is_gimple_builtin_call (call)

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-10 Thread Mike Stump
On Dec 10, 2012, at 12:42 PM, Xinliang David Li wrote: > I have not measured the CFI size impact -- but conceivably it should > be larger -- which is unfortunate. Code speed and size are preferable to optimizing dwarf size… :-) I'd let dwarf 5 fix it!

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-10 Thread Xinliang David Li
I have not measured the CFI size impact -- but conceivably it should be larger -- which is unfortunate. David On Mon, Dec 10, 2012 at 1:23 AM, Richard Biener wrote: > On Sun, Dec 9, 2012 at 2:50 PM, Uros Bizjak wrote: >> Hello! >> >>> I noticed in prologue/epilogue, GCC prefers to use MOVs foll

Re: [patch, mips, testsuite] Fix gcc.target/mips/octeon-bbit-2.c for -Os

2012-12-10 Thread Richard Sandiford
Sorry for the extra-long delay on this one. "Steve Ellcey " writes: > The gcc.target/octeon-bbit-2.c is failing with -Os because that optimization > level does not do whichever optimization it is that results in a bbit instead > of a bbit[01]l. I would like to skip this test for -Os the way it a

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-10 Thread Cary Coutant
> 2012-12-07 Steve Ellcey > > PR target/54061 > rtl.h (IGNORED_DWARF_REGNUM): New. > * dwarf2out.c (reg_loc_descriptor): Check for IGNORED_DWARF_REGNUM. > (mem_loc_descriptor): Ditto. > * config/mips/mips.h (ALL_COP_REG_FIRST): New. > (ALL_COP_REG_

[C++] Add missing dependency on $(PARAMS_H)

2012-12-10 Thread Eric Botcazou
When you add a new --param entry and rebuild an existing compiler, you get weird C++ regressions in the testsuite because of outdated object files. Tested on x86_64-suse-linux, applied on the mainline as obvious. 2012-12-10 Eric Botcazou * Make-lang.in (cp/typeck.o): Add dependency

Re: [patch, mips, testsuite] Fix scan on gcc.target/mips/ext-2.c

2012-12-10 Thread Richard Sandiford
"Steve Ellcey " writes: > In looking at the failure of gcc.target/mips/ext-2.c, it looks like the > GCC compiler is now figuring out that it can use a 32 bit ext instruction > instead of a 64 bit dext instruction and that this is why the test > fails. Here is a patch to look for 'ext' or 'dext',

Re: [PATCH] Bugfix: Additional parameter for canonicalize comparison

2012-12-10 Thread Oleg Endo
On Mon, 2012-12-10 at 11:22 -0800, Richard Henderson wrote: > On 12/10/2012 01:50 AM, Andreas Krebbel wrote: > > Ok. Here is an updated version trying to address the comments above. > > > > 2012-12-10 Andreas Krebbel > > > > * target.def: Define canonicalize_comparison hook. > > * targ

Re: [patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)

2012-12-10 Thread Andrew Pinski
On Mon, Dec 10, 2012 at 9:23 AM, Steve Ellcey wrote: > > The test gcc.target/mips/pr35802.c is failing with the message: > > xgcc: error: unrecognized command line option '-mdsp-mno-shared' > > Obviously the -mdsp and -mno-shared flags got stuck together without > a space between them. Th

Re: [patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)

2012-12-10 Thread Richard Sandiford
"Steve Ellcey " writes: > The test gcc.target/mips/pr35802.c is failing with the message: > > xgcc: error: unrecognized command line option '-mdsp-mno-shared' > > Obviously the -mdsp and -mno-shared flags got stuck together without > a space between them. This patch fixes the failure. Ok t

Ping^2: Synopsys DesignWare ARC Port

2012-12-10 Thread Joern Rennecke
This patch set hasn't been reviewed for more than two weeks: libgcc: 2012-10-09 Joern Rennecke * config.host (arc-*-elf*, arc*-*-linux-uclibc*): New configurations. gcc: 2012-11-22 Joern Rennecke Brendan Kehoe * config.gcc (arc-*-elf*, arc*-*-linux-ucli

Re: [PATCH] Bugfix: Additional parameter for canonicalize comparison

2012-12-10 Thread Richard Henderson
On 12/10/2012 01:50 AM, Andreas Krebbel wrote: > Ok. Here is an updated version trying to address the comments above. > > 2012-12-10 Andreas Krebbel > > * target.def: Define canonicalize_comparison hook. > * targhooks.h (default_canonicalize_comparison): New prototype. > * ta

Re: [patch, testsuite] Fix pr35634 tests when char is unsigned.

2012-12-10 Thread Steve Ellcey
On Mon, 2012-12-10 at 20:01 +0100, Richard Biener wrote: > > Ok minus the initialization. > > Thanks, > Richard. Actually, it looks like you already fixed this last week. I am not seeing it in my git tree, but it is in the FSF subversion and git trees so I think I have a local problem updating

Re: [patch] libgcc unwind-arm.h: mark arguments as unused

2012-12-10 Thread Ian Lance Taylor
On Sun, Dec 9, 2012 at 11:13 PM, Matthias Klose wrote: > This was seen and fixed on trunk building libbacktrace on arm: > http://gcc.gnu.org/ml/gcc-patches/2012-10/msg00628.html > > I see this now cross building libgo targeting arm on the 4.7 branch. Ok to > backport to the 4.7 branch? It's fine

Re: [patch, testsuite] Fix pr35634 tests when char is unsigned.

2012-12-10 Thread Richard Biener
On Mon, Dec 10, 2012 at 7:32 PM, Steve Ellcey wrote: > The tests gcc.dg/torture/pr35634.c and g++.dg/torture/pr35634.C fail for > me on the mips-mti-elf target. At first I thought it was because 'n' > was not initialized but then I realized it was because 'char' on this > target defaults to 'unsi

[patch, testsuite] Fix pr35634 tests when char is unsigned.

2012-12-10 Thread Steve Ellcey
The tests gcc.dg/torture/pr35634.c and g++.dg/torture/pr35634.C fail for me on the mips-mti-elf target. At first I thought it was because 'n' was not initialized but then I realized it was because 'char' on this target defaults to 'unsigned char', not 'signed char'. Since I think initializing 'n'

Re: [patch][c++] Stop in cp_write_global_declarations after writing a PCH

2012-12-10 Thread Steven Bosscher
On Fri, Dec 7, 2012 at 3:13 PM, Richard Biener wrote: > On Fri, Dec 7, 2012 at 10:52 AM, Steven Bosscher wrote: >> Ping? > > Ok if C++ frontend maintainers don't complain over the weekend. Committed now. BTW I forgot to mention I also did a build+test with this patch and with gcac checking enabled

Re: application/xml mime-type in recent libstdc++ doc changes

2012-12-10 Thread Benjamin De Kosnik
> > libstdc++-v3/doc/xsl/customization.xsl.in is marked as > > > > svn:mime-type = application/xml > > > > at least on the 4.7 branch, having some unexpected outcome for svn > > diff. If this was unintended, could you change the svn:mime-type > > back to text? This is unintentional, and frank

[patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)

2012-12-10 Thread Steve Ellcey
The test gcc.target/mips/pr35802.c is failing with the message: xgcc: error: unrecognized command line option '-mdsp-mno-shared' Obviously the -mdsp and -mno-shared flags got stuck together without a space between them. This patch fixes the failure. Ok to checkin? It was tested with no

[patch, mips, testsuite] Fix scan on gcc.target/mips/ext-2.c

2012-12-10 Thread Steve Ellcey
In looking at the failure of gcc.target/mips/ext-2.c, it looks like the GCC compiler is now figuring out that it can use a 32 bit ext instruction instead of a 64 bit dext instruction and that this is why the test fails. Here is a patch to look for 'ext' or 'dext', OK to checkin? It fixes the failu

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-10 Thread Steve Ellcey
On Mon, 2012-12-10 at 10:53 +, Richard Sandiford wrote: > So please ignore that stupid suggestion. The comments about the MIPS > parts still stand of course. > > Richard Here is a new patch. I made the MIPS changes and I also changed some code in dwarf2out.c. I was making some calls to db

Re: [PING][PATCH][RFC] Bug handling SUBREG (MEM) - MEM having side-effects?

2012-12-10 Thread Tejas Belagod
PING. Tejas Belagod wrote: Richard Sandiford wrote: After the discussions Richard Earnshaw had on IRC with Andrew Pinski, it was felt that it was best to fix the standard predicate 'general_operand' to not allow SUBREG (MEM) with side-effects as it has known issues associated with it - particul

Re: [AARCH64-4.7, PATCH, committed] Fix race in parallel build.

2012-12-10 Thread Richard Earnshaw
On 21/11/12 13:31, Marcus Shawcroft wrote: I've just committed the attached back port of: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01775.html to the ARM/aarch64-4.7-branch. /Marcus 2012-11-21 Marcus Shawcroft * Makefile.in (gengtype-lex.o): Add dependency on $(BCONFIG_H). ba

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread David Edelsohn
On Mon, Dec 10, 2012 at 10:54 AM, Jakub Jelinek wrote: > On Mon, Dec 10, 2012 at 10:49:57AM -0500, David Edelsohn wrote: >> On Mon, Dec 10, 2012 at 5:05 AM, Jakub Jelinek wrote: >> > On Mon, Dec 10, 2012 at 10:56:58AM +0100, Janus Weil wrote: >> >> > For C and C++, identifiers beginning with unde

[google/gcc-4_7] Backport r193747 to using working set for bb hotness (issue6907063)

2012-12-10 Thread Teresa Johnson
This is a backport of r193747 to use the working set from the profile summary to determine the bb hot count threshold. Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for google 4_7 branch? Thanks, Teresa 2012-12-10 Teresa Johnson Backport r193747 from trunk: 2012-11

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread Jakub Jelinek
On Mon, Dec 10, 2012 at 10:49:57AM -0500, David Edelsohn wrote: > On Mon, Dec 10, 2012 at 5:05 AM, Jakub Jelinek wrote: > > On Mon, Dec 10, 2012 at 10:56:58AM +0100, Janus Weil wrote: > >> > For C and C++, identifiers beginning with underscore and upper case > >> > letter or with two underscores a

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread David Edelsohn
On Mon, Dec 10, 2012 at 5:05 AM, Jakub Jelinek wrote: > On Mon, Dec 10, 2012 at 10:56:58AM +0100, Janus Weil wrote: >> > For C and C++, identifiers beginning with underscore and upper case >> > letter or with two underscores are reserve to the implementation. >> >> See the previous discussion in t

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread David Edelsohn
On Mon, Dec 10, 2012 at 4:22 AM, Janus Weil wrote: > 2012/12/10 David Edelsohn : >> There has been no progress on this bug for over two months. I have >> opened PR fortran/55636. This must be fixed before GCC 4.8 ships. > > Thanks for pinging, David. I almost forgot about it ... > > Here is a re

[PATCH] Fix PR55079

2012-12-10 Thread Richard Biener
This improves VRP so that two of the three (the remaining one invalid IMHO) spurious array-bound warnings in PR55079 no longer appear. With the patch we now properly handle MIN_EXPR <[1, 6], VARYING> and register asserts for i_2 in i_2 = i_1 + 1; if (i_1 != 0) which is a common pattern f

Re: PATCH: PR lto/55466: Revision 191466 destroyed DWARF debug info

2012-12-10 Thread H.J. Lu
On Mon, Dec 10, 2012 at 1:09 AM, Richard Biener wrote: > On Fri, Dec 7, 2012 at 11:59 PM, H.J. Lu wrote: >> Hi, >> >> Since revision 191466, LTO no longer maintains its own symbol table and >> it uses varpool for variables instead. As the result, debug info on >> variables is lost when GCC is co

Re: [patch] don't build multilib libraries during bootstrap, and disable some libstdc++ features

2012-12-10 Thread H.J. Lu
On Mon, Dec 10, 2012 at 5:53 AM, Matthias Klose wrote: > Am 10.12.2012 13:16, schrieb Matthias Klose: >> During bootstrap some things are built which are not required for the >> bootstrap: >> >> - multilib libraries >> - libstdc++ debug library, when configured with --enable-libstdcxx-debug >>

[PATCH] Fix PR55107

2012-12-10 Thread Richard Biener
This fixes PR55107 - the way we dealt with fully redundant values found by PRE phi-translation breaks existing SETs, so don't special case it in a hacky way but instead create a simple assignment. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-12-10 Richar

Re: [patch] don't build multilib libraries during bootstrap, and disable some libstdc++ features

2012-12-10 Thread Matthias Klose
Am 10.12.2012 13:16, schrieb Matthias Klose: > During bootstrap some things are built which are not required for the > bootstrap: > > - multilib libraries > - libstdc++ debug library, when configured with --enable-libstdcxx-debug > - libstdc++ precompiled header files > > The attached patch d

Re: [patch] don't build multilib libraries during bootstrap, and disable some libstdc++ features

2012-12-10 Thread Richard Biener
On Mon, Dec 10, 2012 at 1:18 PM, Andrew Pinski wrote: > On Mon, Dec 10, 2012 at 4:16 AM, Matthias Klose wrote: >> During bootstrap some things are built which are not required for the >> bootstrap: >> >> - multilib libraries >> - libstdc++ debug library, when configured with --enable-libstdcxx

Re: [PATCH] SPU: Fix build failure on 64 bit hosts

2012-12-10 Thread Richard Biener
On Mon, Dec 10, 2012 at 12:31 PM, Andreas Krebbel wrote: > Hi, > > building an SPU compiler on a 64 bit host system currently fails with: > > spu.md:4394: error: cast from ‘rtx_def**’ to ‘int’ loses precision > > In the SPU backend several patterns mention the operands array in the > insn conditio

Re: Fix ICE on loop over constant vector at -O

2012-12-10 Thread Richard Biener
On Mon, Dec 10, 2012 at 10:55 AM, Eric Botcazou wrote: >> Well ... I would have expected that we'd have folded the CONST_DECL to >> its DECL_INITIAL. At least that is what we do for all other CONST_DECLs. >> The only way CONST_DECLs should appear in the IL (after some optimization >> of course) i

[asan, committed] Sync asan_test.cc from upstream

2012-12-10 Thread Jakub Jelinek
Hi! I've committed as obvious this change to sync the test from upstream to avoid one unnecessary FAIL. 2012-12-10 Jakub Jelinek * g++.dg/asan/asan_test.cc: Sync from upstream. * g++.dg/asan/asan_test_utils.h: Likewise. --- gcc/testsuite/g++.dg/asan/asan_test.cc 2012-12-

Re: [patch] don't build multilib libraries during bootstrap, and disable some libstdc++ features

2012-12-10 Thread Andrew Pinski
On Mon, Dec 10, 2012 at 4:16 AM, Matthias Klose wrote: > During bootstrap some things are built which are not required for the > bootstrap: > > - multilib libraries > - libstdc++ debug library, when configured with --enable-libstdcxx-debug > - libstdc++ precompiled header files > > The attache

[patch] don't build multilib libraries during bootstrap, and disable some libstdc++ features

2012-12-10 Thread Matthias Klose
During bootstrap some things are built which are not required for the bootstrap: - multilib libraries - libstdc++ debug library, when configured with --enable-libstdcxx-debug - libstdc++ precompiled header files The attached patch disables building these during the bootstrap stages. The additi

Re: [asan] Instrument STRING_CSTs

2012-12-10 Thread Dodji Seketeli
Jakub Jelinek writes: > 2012-12-05 Jakub Jelinek > > * asan.c (asan_init_shadow_ptr_types): Move earlier in the file. > Call initialize_sanitizer_builtins at the end. > (asan_pp_string): Use TREE_TYPE (shadow_ptr_types[0]) > as character type instead of char_type_node.

[PATCH] SPU: Fix build failure on 64 bit hosts

2012-12-10 Thread Andreas Krebbel
Hi, building an SPU compiler on a 64 bit host system currently fails with: spu.md:4394: error: cast from ‘rtx_def**’ to ‘int’ loses precision In the SPU backend several patterns mention the operands array in the insn condition in order to prevent a warning about missing modes to be issued. Inte

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-10 Thread Richard Sandiford
"H.J. Lu" writes: > On Sat, Dec 8, 2012 at 2:14 AM, Richard Sandiford > wrote: >> >> As far as the dwarf2out.c bits go, I think the original dbx_reg_number >> assert: >> >> gcc_assert (regno != INVALID_REGNUM); >> >> should become: >> >> gcc_assert (regno != INVALID_REGNUM && regno != IGNORE

Re: [RFC] Remove TLS symbols from section anchor blocks

2012-12-10 Thread Richard Sandiford
David Edelsohn writes: > [Sorry, I forgot to copy GCC Patches before] > > Some of the libgomp testcases fail on AIX when using native TLS > because variables annotated with > > #pragma omp threadprivate(thr) > > sometimes are placed in section anchor blocks. Variables declared > with "__thread" c

Re: [PATCH][ARM][3/3] AArch32 NEON vrint builtins and intrinsics

2012-12-10 Thread Ramana Radhakrishnan
On 11/29/12 14:27, Kyrylo Tkachov wrote: Hi all, This patch adds the intrinsics support for the vrnd intrinsics that are implemented by the vrint instructions. The .ml scripts contain the new information and should used to regenerate the arm_neon.h header file, tests and documentation. In particu

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread Jakub Jelinek
On Mon, Dec 10, 2012 at 10:56:58AM +0100, Janus Weil wrote: > > For C and C++, identifiers beginning with underscore and upper case > > letter or with two underscores are reserve to the implementation. > > See the previous discussion in this thread ... That's true, but the implementation isn't ju

Re: Fix ICE on loop over constant vector at -O

2012-12-10 Thread Eric Botcazou
> Well ... I would have expected that we'd have folded the CONST_DECL to > its DECL_INITIAL. At least that is what we do for all other CONST_DECLs. > The only way CONST_DECLs should appear in the IL (after some optimization > of course) is when you take their address. That's what (essentially) ha

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread Janus Weil
>> Here is a re-diffed version of my previous patch. I'd be grateful for >> comments. In particular: Does it fix all the problems on AIX? (For >> more open questions, see below.) > > IMHO there is no point to mangle automatic vars or PARM_DECLs that way, > i.e. anything that won't be TREE_STATIC.

Re: [PATCH] Bugfix: Additional parameter for canonicalize comparison

2012-12-10 Thread Andreas Krebbel
On Sat, Dec 08, 2012 at 01:16:16AM +0100, Oleg Endo wrote: > On Fri, 2012-12-07 at 13:37 -0600, Richard Henderson wrote: > > On 2012-12-06 02:25, Andreas Krebbel wrote: > > > ! targetm.canonicalize_comparison ((int*)&compare_code, &op0, > > > &op1, 1); > > > > The basic approach seems s

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread Jakub Jelinek
On Mon, Dec 10, 2012 at 10:22:30AM +0100, Janus Weil wrote: > 2012/12/10 David Edelsohn : > > There has been no progress on this bug for over two months. I have > > opened PR fortran/55636. This must be fixed before GCC 4.8 ships. > > Thanks for pinging, David. I almost forgot about it ... > >

Re: [Patch, Fortran, F03] PR52909: Procedure pointers not private to modules

2012-12-10 Thread Tobias Burnus
Hi Janus, I think the patch below is okay, especially in light of patch PR49110/51055 which also contains a slight ABI breakage. Can you mention the potential ABI issue in the release notes (gfortran wiki). Tobias On 09.04.2012 18:31 Janus Weil wrote: Hi all, here is a simple patch for a p

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-10 Thread Richard Biener
On Sun, Dec 9, 2012 at 2:50 PM, Uros Bizjak wrote: > Hello! > >> I noticed in prologue/epilogue, GCC prefers to use MOVs followed by a >> SP adjustment instead of a sequence of pushes/pops. The preference to >> the MOVs are good for old CPU micro-architectures (before pentium-4, >> K10), because i

Re: [Fortran, (RFC) patch] PR49110/51055 Assignment to alloc. deferred-length character vars

2012-12-10 Thread Janus Weil
2012/12/10 David Edelsohn : > There has been no progress on this bug for over two months. I have > opened PR fortran/55636. This must be fixed before GCC 4.8 ships. Thanks for pinging, David. I almost forgot about it ... Here is a re-diffed version of my previous patch. I'd be grateful for comm

Re: [patch print-tree.c]: 5 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-12-10 Thread Richard Biener
On Sun, Dec 9, 2012 at 5:18 PM, NightStrike wrote: > On Sat, Dec 1, 2012 at 11:12 PM, Kai Tietz wrote: >> Ping >> >> 2012/11/29 Kai Tietz : >>> Hello, >>> >>> this trivial patch fixes a bootstrap issue on LLP64 hosts. >>> >>> ChangeLog >>> >>> 2012-11-29 Kai Tietz >>> >>> PR target/53912

Re: [patch stmt.c]: Fix SjLj exception handling

2012-12-10 Thread Richard Biener
On Sun, Dec 9, 2012 at 5:17 PM, NightStrike wrote: > On Sat, Dec 1, 2012 at 12:28 PM, Steven Bosscher > wrote: >> On Sat, Dec 1, 2012 at 10:59 PM, Kai Tietz wrote: >>> Hi, >>> >>> recent 4,8 has regressions in g++.old-deja/g++.eh for the catch*.C >>> tests, if exception-mechanism is SjLj. This

Re: [patch stmt.c]: 6 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

2012-12-10 Thread Richard Biener
On Sun, Dec 9, 2012 at 4:58 PM, NightStrike wrote: > On Sat, Dec 1, 2012 at 11:13 PM, Kai Tietz wrote: >> Ping >> >> 2012/11/29 Kai Tietz : >>> Hello, >>> >>> this trivial patch fixes a bootstrap issue on LLP64 hosts. >>> >>> ChangeLog >>> >>> 2012-11-29 Kai Tietz >>> >>> PR target/53912

Re: Fix ICE on loop over constant vector at -O

2012-12-10 Thread Richard Biener
On Sat, Dec 8, 2012 at 12:42 PM, Eric Botcazou wrote: > This is an internal error in for_each_index at -O: > > +===GNAT BUG DETECTED==+ > | 4.8.0 20121208 (experimental) [trunk revision 194319] (x86_64-suse-linux) > GCC error:| > | in for_each_in

Re: PATCH: PR lto/55466: Revision 191466 destroyed DWARF debug info

2012-12-10 Thread Richard Biener
On Fri, Dec 7, 2012 at 11:59 PM, H.J. Lu wrote: > Hi, > > Since revision 191466, LTO no longer maintains its own symbol table and > it uses varpool for variables instead. As the result, debug info on > variables is lost when GCC is configured with > > --enable-bootstrap --with-build-config=bootst

Re: [patch] [libstdc++] Fix build failure with --enable-libstdcxx-debug

2012-12-10 Thread Jonathan Wakely
On 10 December 2012 08:22, Matthias Klose wrote: > Am 10.12.2012 08:19, schrieb Matthias Klose: >> Seen with a x86_64 x arm x arm cross build and --enable-libstdcxx-debug. The >> ../config.h should not be hard-coded. Using the macro guarantees that the >> rewritten macros for the debug builds are

Re: [patch] [libstdc++] Fix build failure with --enable-libstdcxx-debug

2012-12-10 Thread Matthias Klose
Am 10.12.2012 08:19, schrieb Matthias Klose: > Seen with a x86_64 x arm x arm cross build and --enable-libstdcxx-debug. The > ../config.h should not be hard-coded. Using the macro guarantees that the > rewritten macros for the debug builds are used. However I fail to see why the > unpatched version