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

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

2012-12-10 Thread Jonathan Wakely
On 10 December 2012 08:22, Matthias Klose d...@ubuntu.com 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

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 hongjiu...@intel.com 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

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 ebotca...@adacore.com 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:|

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 nightstr...@gmail.com wrote: On Sat, Dec 1, 2012 at 11:13 PM, Kai Tietz ktiet...@googlemail.com wrote: Ping 2012/11/29 Kai Tietz ktiet...@googlemail.com: Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29

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

2012-12-10 Thread Richard Biener
On Sun, Dec 9, 2012 at 5:17 PM, NightStrike nightstr...@gmail.com wrote: On Sat, Dec 1, 2012 at 12:28 PM, Steven Bosscher stevenb@gmail.com 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

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 nightstr...@gmail.com wrote: On Sat, Dec 1, 2012 at 11:12 PM, Kai Tietz ktiet...@googlemail.com wrote: Ping 2012/11/29 Kai Tietz ktiet...@googlemail.com: Hello, this trivial patch fixes a bootstrap issue on LLP64 hosts. ChangeLog 2012-11-29

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

2012-12-10 Thread Janus Weil
2012/12/10 David Edelsohn dje@gmail.com: 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

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 ubiz...@gmail.com 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),

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

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 dje@gmail.com: 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

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. And,

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)

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 just

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

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

2012-12-10 Thread Richard Sandiford
H.J. Lu hjl.to...@gmail.com writes: On Sat, Dec 8, 2012 at 2:14 AM, Richard Sandiford rdsandif...@googlemail.com 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 !=

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

Re: [asan] Instrument STRING_CSTs

2012-12-10 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com writes: 2012-12-05 Jakub Jelinek ja...@redhat.com * 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

[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

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 d...@ubuntu.com 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

[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 ja...@redhat.com * g++.dg/asan/asan_test.cc: Sync from upstream. * g++.dg/asan/asan_test_utils.h: Likewise. ---

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 ebotca...@adacore.com 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

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 kreb...@linux.vnet.ibm.com 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

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 pins...@gmail.com wrote: On Mon, Dec 10, 2012 at 4:16 AM, Matthias Klose d...@ubuntu.com wrote: During bootstrap some things are built which are not required for the bootstrap: - multilib libraries - libstdc++ debug library, when configured

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 disables

[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

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 d...@ubuntu.com 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

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 richard.guent...@gmail.com wrote: On Fri, Dec 7, 2012 at 11:59 PM, H.J. Lu hongjiu...@intel.com 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

[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

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 ja...@gcc.gnu.org wrote: 2012/12/10 David Edelsohn dje@gmail.com: 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

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 ja...@redhat.com 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

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 ja...@redhat.com 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

[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 tejohn...@google.com Backport r193747 from

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 ja...@redhat.com 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 ja...@redhat.com wrote: On Mon, Dec 10, 2012 at 10:56:58AM +0100, Janus Weil wrote: For C and C++,

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 marcus.shawcr...@arm.com * Makefile.in (gengtype-lex.o): Add

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 -

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

[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

[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

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 frankly I don't

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.

[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

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 sell...@mips.com 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

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 d...@ubuntu.com 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?

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 my

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 andreas.kreb...@de.ibm.com * target.def: Define canonicalize_comparison hook. * targhooks.h (default_canonicalize_comparison): New

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 joern.renne...@embecosm.com * config.host (arc-*-elf*, arc*-*-linux-uclibc*): New configurations. gcc: 2012-11-22 Joern Rennecke joern.renne...@embecosm.com Brendan Kehoe

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

2012-12-10 Thread Richard Sandiford
Steve Ellcey sell...@mips.com 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

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 andreas.kreb...@de.ibm.com * target.def: Define canonicalize_comparison

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

2012-12-10 Thread Richard Sandiford
Steve Ellcey sell...@mips.com 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'

[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 ebotca...@adacore.com * Make-lang.in

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

2012-12-10 Thread Cary Coutant
2012-12-07 Steve Ellcey sell...@mips.com 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.

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 sell...@mips.com 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

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 richard.guent...@gmail.com wrote: On Sun, Dec 9, 2012 at 2:50 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! I noticed in

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 davi...@google.com 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: [asan] Handle noreturn calls with __asan_handle_no_return ()

2012-12-10 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com 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

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 ja...@redhat.com 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

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; +}

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 sell...@mips.com 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

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

2012-12-10 Thread Kaz Kojima
Andreas Krebbel kreb...@linux.vnet.ibm.com 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, 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:31

[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

[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

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 de...@google.com 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

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 de...@google.com 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

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 pins...@gmail.com wrote: On Mon, Dec 10, 2012 at 9:23 PM, Dehao Chen de...@google.com 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

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 back to text?

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 jin...@google.com *