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
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
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
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
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
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
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
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
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)
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,
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
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
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
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)
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.
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
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_
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
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.
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
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)
OK.
Jason
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
23 matches
Mail list logo