Committed: Fix PR53156, testsuite regression from recent reload change

2012-04-29 Thread Hans-Peter Nilsson
The regression was mostly due to a failed assumption by the test-case (a minimal solution must match the assembly code patterns), but also due to a different suboptimal sequence after the reload change. The committed patch below fixes the regresssed code, the test-case, and tweaks the comment to m

[patch, fortran, committed] Fix PR 53148

2012-04-29 Thread Thomas Koenig
Hello world, I have committed the attached patch as obvious as rev. 186942 after regression-testing. Will also commit to 4.7 in a few days. Thomas 2012-04-29 Thomas Koenig PR fortran/53148 * frontend-passes.c (create_var): If the statement has a label, put

[committed] Make Wmissing-noreturn an alias of -Wsuggest-attribute=noreturn.

2012-04-29 Thread Manuel López-Ibáñez
Bootstrapped and regression tested. Committed as obvious at revision 186943. Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 186942) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,8 @@ +2012-04-29 Manuel López-Ibáñez

[c/c++] Alias Wmissing-format-attribute to Wsuggest-attribute=format

2012-04-29 Thread Manuel López-Ibáñez
Since Wsuggest-attribute=[pure|const|noreturn] was introduced, it makes sense to have Wsuggest-attribute=format. This patch makes this the canonical form of Wmissing-format-attribute (the old form is still accepted and documented). Bootstrapped and regression tested. OK? 2012-04-29 Manuel Lópe

[committed] adjust testcase gcc.dg/20011021-1.c

2012-04-29 Thread Manuel López-Ibáñez
An oversight from a previous commit. Committed as obvious after testing. Cheers, Manuel. Index: gcc/testsuite/gcc.dg/20011021-1.c === --- gcc/testsuite/gcc.dg/20011021-1.c (revision 186942) +++ gcc/testsuite/gcc.dg/20011021-1.c

Re: [libcpp] maybe canonicalize system paths in line-map

2012-04-29 Thread Manuel López-Ibáñez
New version of the patch. Bootstrapped and regression tested. Is this version OK? 2012-04-29 Manuel López-Ibáñez Dodji Seketeli PR 5297 * libcpp/files.c (maybe_shorter_path): New. (find_file_in_dir): Use it. pr52974.diff Description: Binary data

Re: [RFC] improve caret diagnostics for overload failures

2012-04-29 Thread Manuel López-Ibáñez
A new version using unsigned int for the flag type. It also adds another use in the C FE. I am not asking for approval, only whether this approach/implementation is the way to go. Cheers, Manuel. On 23 April 2012 20:09, Manuel López-Ibáñez wrote: > So, apart from the type of the flag, are the

Re: [v3] is_modulo

2012-04-29 Thread Gabriel Dos Reis
On Sat, Apr 28, 2012 at 6:45 PM, Marc Glisse wrote: > On Sat, 28 Apr 2012, Gabriel Dos Reis wrote: > >> On Sat, Apr 28, 2012 at 5:41 PM, Marc Glisse wrote: >>> >>> Hello, >>> >>> the attached follows the precisions on the definition of is_modulo in DR >>> 612. I believe this is what the values al

Re: [c/c++] Alias Wmissing-format-attribute to Wsuggest-attribute=format

2012-04-29 Thread Gabriel Dos Reis
On Sun, Apr 29, 2012 at 4:28 AM, Manuel López-Ibáñez wrote: > Since Wsuggest-attribute=[pure|const|noreturn] was introduced, it > makes sense to have Wsuggest-attribute=format. This patch makes this > the canonical form of Wmissing-format-attribute (the old form is still > accepted and documented)

Re: [RFH / Patch] PR 51222

2012-04-29 Thread Paolo Carlini
Hi, On 04/29/2012 05:17 AM, Jason Merrill wrote: On 04/28/2012 07:12 AM, Paolo Carlini wrote: isn't, but clearly can be instantiation dependent. Then, I guess the way I'm proposing to handle this is by starting some sort of embryonic instantiation_dependent_expression_p and using it only here,

[PATCH, libcpp] Don't use C++ style comments in libcpp

2012-04-29 Thread Dodji Seketeli
Hello, I noticed that the file lex.c had C++ style comments, which I believe is against the coding standards of the project. Fixed, tested and applied to the mainline as per the obvious rule. libcpp/ * lex.c (lex_raw_string): Change C++ style comments into C style commen

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-04-29 Thread Pascal Obry
Arno, > Pascal, I'd suggest you go ahead and revert this patch. This patch has already been reverted by Eric on Apr 10 on gcc-4.7 branch. -- Pascal Obry -- gpg --keyserver keys.gnupg.net --recv-key F949BD3B

Re: [PATCH 04/11] Fix expansion point loc for macro-like tokens

2012-04-29 Thread Dodji Seketeli
Jason Merrill writes: > On 04/25/2012 05:07 AM, Dodji Seketeli wrote: >> + /* If the first token we got was a padding token, let's put >> + it back into the stream so that cpp_get_token will get it >> + first; and if we are currently expanding a macro, don't >> + forg

Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-29 Thread Dodji Seketeli
Jason Merrill writes: > On 04/25/2012 11:31 AM, Dodji Seketeli wrote: >> +#define EXPANSION_POINT_LOCATION_FILE(LOC) \ >> + ((expand_location_to_expansion_point (LOC)).file) >> +#define EXPANSION_POINT_LOCATION_LINE(LOC) \ >> + ((expand_location_to_expansion_point (LOC)).line)

Re: [RFH / Patch] PR 51222

2012-04-29 Thread Paolo Carlini
On 04/29/2012 05:42 PM, Paolo Carlini wrote: Otherwise, I'm attaching something very close to the letter of the ABI, which passes as-is the testsuite. Which in any case should use cp_tree_operand_length. Paolo.

Re: [libcpp] maybe canonicalize system paths in line-map

2012-04-29 Thread Dodji Seketeli
Manuel López-Ibáñez a écrit: > PR 5297 > * libcpp/files.c (maybe_shorter_path): New. > (find_file_in_dir): Use it. I can't approve or deny this patch, but for what it's worth, I find it fine. Thanks. -- Dodji

PATCH: Add x32 support to boehm-gc

2012-04-29 Thread H.J. Lu
Hi, This patch adds x32 support to boehm-gc. The same patch has been sent to the boehm-gc mailing list. Tested on Linux/x32 and Linux/x86-64. OK for trunk? Thanks. H.J. - Forwarded message from "H.J. Lu" - Date: Mon, 16 Apr 2012 09:39:20 -0700 From: "H.J. Lu" To: "H.J. Lu" Cc: hans_

Re: [PATCH 12/13] Adjust relevant test cases wrt -ftrack-macro-expansion=[0|2]

2012-04-29 Thread Dodji Seketeli
Gabriel Dos Reis writes: > OK. Thank you. While bootstrapping the tree again, it appeared that an output regression of the objc test objc.dg/foreach-7.m flew below my radar. It's one of those typical cases where the first location pointed to by the diagnostics points into the definition of a m

[v3] constrain std::function constructor to only accept callable types

2012-04-29 Thread Jonathan Wakely
Currently we only constrain std::function's constructor to reject integral arguments, this patch changes it to reject non-callable arguments. This is the proposed resolution of LWG 2132, I had already planned to do this anyway before the issue was opened so I don't see any need to wait for a DR.

[Patch ping] Strength reduction

2012-04-29 Thread William J. Schmidt
Thought I'd ping http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01225.html since it's been about six weeks. Any initial feedback would be very much appreciated! Thanks, Bill

[v3] Completely fix libstdc++/51795 in mainline

2012-04-29 Thread Paolo Carlini
Hi, contributed by Marc (thanks Again!) and integrated by me. Tested x86_64-linux, multilib. Committed to mainline. Thanks, Paolo. /// 2012-04-29 Marc Glisse Paolo Carlini PR libstdc++/51795 * include/bits/stl_algobase.h (__lg<>(_Size)

Re: [PATCH 04/11] Fix expansion point loc for macro-like tokens

2012-04-29 Thread Jason Merrill
OK. Jason

Re: [PATCH 12/13] Adjust relevant test cases wrt -ftrack-macro-expansion=[0|2]

2012-04-29 Thread Dodji Seketeli
Dodji Seketeli writes: > Gabriel Dos Reis writes: > >> OK. > > Thank you. > > While bootstrapping the tree again, it appeared that an output > regression of the objc test objc.dg/foreach-7.m flew below my radar. > > It's one of those typical cases where the first location pointed to by > the dia