Re: [google] change dump_inline_decisions to make it print more useful and better looking info

2011-12-31 Thread Xinliang David Li
Ok for google branches. David On Sat, Dec 31, 2011 at 12:45 AM, Dehao Chen wrote: > I see. The patch is updated: > > Thanks, > Dehao > > Index: ipa-inline.c > === > --- ipa-inline.c        (revision 182739) > +++ ipa-inline.c      

Re: [PATCH SMS 2/2, RFC] Register pressure estimation for the partial schedule (re-submission)

2011-12-31 Thread Ayal Zaks
On Mon, Dec 19, 2011 at 4:28 PM, Richard Sandiford wrote: > Hi Revital, > > Revital Eres writes: >> The attached patch is a resubmission following comments made by Ayal >> and Richard. >> >> Tested and bootstrap with the other patches in the series on >> ppc64-redhat-linux, enabling SMS on loops

Re: [C++ Patch] PR 51379

2011-12-31 Thread Paolo Carlini
Hi, On 12/31/2011 03:08 PM, Paolo Carlini wrote: this implements in a straightforward way the resolution of DR 799 (which went in CD2): basically a reinterpret_cast to the type of the expression itself is fine for integral, enumeration, pointer, or pointer-to-member type. Tested x86_64-linux.

Re: [C++ Patch] PR 51379

2011-12-31 Thread Jason Merrill
On 12/31/2011 03:08 PM, Paolo Carlini wrote: this implements in a straightforward way the resolution of DR 799 (which went in CD2): basically a reinterpret_cast to the type of the expression itself is fine for integral, enumeration, pointer, or pointer-to-member type. Tested x86_64-linux. + e

Re: [C++ Patch] PR 51379

2011-12-31 Thread Jason Merrill
OK. Jason

[C++ Patch] PR 51379

2011-12-31 Thread Paolo Carlini
Hi, this implements in a straightforward way the resolution of DR 799 (which went in CD2): basically a reinterpret_cast to the type of the expression itself is fine for integral, enumeration, pointer, or pointer-to-member type. Tested x86_64-linux. Thanks, Paolo. / /cp 2

Re: Keep static VTA locs in cselib tables only

2011-12-31 Thread Alexandre Oliva
On Nov 25, 2011, Jakub Jelinek wrote: > The numbers I got with your patch (RTL checking) are below, seems > the cumulative numbers other than 100% are all bigger with patched stage2, > which means unfortunately debug info quality degradation. Not really. I found some actual degradation after fi

Re: [C++ Patch] deprecation of access declarations

2011-12-31 Thread Mike Stump
On Dec 31, 2011, at 9:14 AM, Jason Merrill wrote: > On 12/30/2011 04:34 PM, Mike Stump wrote: >> So, I'm wondering why it was done this way originally: >> >> - static int S1; // ERROR - uses same name 9.3 >> + static int S1; // { dg-error "" } uses same name 9.3 >> >

[v3] proposed fix for libstdc++/49204 causes abi_check failure

2011-12-31 Thread Jonathan Wakely
I want to commit the attached patch: PR libstdc++/49204 * include/std/future (__future_base::_State_base::wait()): Call _M_complete_async instead of _M_run_deferred. (__future_base::_State_base::wait_for()): Call _M_has_deferred and return future_status.

[Patch, fortran] PR fortran/50981 segmentation fault when trying to access absent elemental actual arg

2011-12-31 Thread Mikael Morin
Hello, as promised, here is a fix for pr50981. Currently, for a call to an elemental procedure, every scalar actual argument is evaluated before the loop containing the function call. The bug is, we can't evaluate the actual argument if it is a reference to an absent optional dummy argument, as it

Re: [C++ Patch] PR 51397

2011-12-31 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] deprecation of access declarations

2011-12-31 Thread Jason Merrill
On 12/30/2011 04:34 PM, Mike Stump wrote: So, I'm wondering why it was done this way originally: - static int S1; // ERROR - uses same name 9.3 + static int S1; // { dg-error "" } uses same name 9.3 and not: - static int S1; // ERROR - uses same nam

Re: RE :Re: RE :Re: hashtable local iterator

2011-12-31 Thread Jonathan Wakely
On 29 December 2011 18:00, François Dumont wrote: > Attached patch applied. Thanks! >        (_Local_iterator_base<>, _Locale_iterator<>, > _Locale_const_iterator<>): I've fixed these two instances of "Locale" in the ChangeLog I might also rename _Ebo_helper to something more specific such as

[Patch, Fortran] PR 51682 - Fix coarray issues with -fdefault-integer-8

2011-12-31 Thread Tobias Burnus
Dear all, first, I hope you will have (or already had) a good start into the new year! This patch is rather obvious. Thanks to Dominique for reporting the issue. Build and regtested on x86-64-linux. OK for the trunk? Happy New Year to all, Tobias caf-i8.diff Description: application/unknow

[C++ Patch] PR 51397

2011-12-31 Thread Paolo Carlini
Hi, this issue, reported by Dave (and Jon), is about the error message printed for a static_assert like: static_assert('X' != '\130', "'X' has the wrong value"); where, due to the use of %E, we go through pp_c_string_literal and thus pp_c_char, and we escape the single quotes around X. D

Re: [google] change dump_inline_decisions to make it print more useful and better looking info

2011-12-31 Thread Dehao Chen
I see. The patch is updated: Thanks, Dehao Index: ipa-inline.c === --- ipa-inline.c(revision 182739) +++ ipa-inline.c(working copy) @@ -303,26 +303,19 @@ char *buf; size_t buf_size; const char *bfd_name = lan

Re: [Patch, fortran] PR46328 - [OOP] type-bound operator call with non-trivial polymorphic operand

2011-12-31 Thread Paul Richard Thomas
Dear Tobias, Many thanks for the review and your replies to redux(1)/(2). I will take account of your comments and commit tomorrow night. I have a fix for the allocation of class temporaries, which cures the problem with index vectors It will see the light of day on Monday. Happy New Year t