public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Bernd Schmidt <bernds_cb1@t-online.de>
To: Kenneth Zadeck <zadeck@naturalbridge.com>
Cc: gcc-patches <gcc-patches@gcc.gnu.org>,
	  Steven Bosscher <stevenb.gcc@gmail.com>,
	 "Park, Seongbae" <seongbae.park@gmail.com>,
	  "Bonzini, Paolo" <bonzini@gnu.org>,
	 Serge Belyshev <belyshev@depni.sinp.msu.ru>,
	 richard.earnshaw@arm.com,   echristo@apple.com,   "Pinski,
	Andrew" <andrew_pinski@playstation.sony.com>,
	 "Weigand, Ulrich" <Ulrich.Weigand@de.ibm.com>,
	  Ian Lance Taylor <iant@google.com>,
	 "Edelsohn, David" <dje@watson.ibm.com>,
	  "Berlin, Daniel" <dberlin@dberlin.org>
Subject: Re: dataflow branch merging plans.
Date: Fri, 25 May 2007 09:51:00 -0000	[thread overview]
Message-ID: <4656AFFD.1040605@t-online.de> (raw)
In-Reply-To: <4655EF12.6060605@naturalbridge.com>

Kenneth Zadeck wrote:
> Bernd Schmidt wrote:
>> Kenneth Zadeck wrote:
>>   
>>> I believe that the dataflow branch is now ready to merge into the
>>> mainline.  We have fixed almost all of the performance problems
>>> associated with it.  While there are still some left, we feel
>>> confident that these can be addressed during the rest of stage I and
>>> during stage II.
>>>     
>> Vlad's last benchmark run still showed up to 11% compile time
>> regression, didn't it?
>>
>>
>> Bernd
>>   
> The ia-64 is an outlier at 11% as is the ppc at .9%.  I think that it is
> quite likely that there is some ia-64 unique pass, like one of the
> schedulers, that needs a look at.  This will be addressed after the merge. 

Still, 6% compile time regression on several targets and typically a
(very small) regression in SPEC scores - am I the only one who's not
impressed?  We don't normally accept patches with these kinds of
results, and I don't see why we should make an exception here.

Why the rush to merge now?  How exactly will it make 4.3 better?

Finally, since a plan to merge next week was indicated - which
maintainer has approved it?


Bernd
-- 
This footer brought to you by insane German lawmakers.
Analog Devices GmbH      Wilhelm-Wagenfeld-Str. 6      80807 Muenchen
Sitz der Gesellschaft Muenchen, Registergericht Muenchen HRB 40368
Geschaeftsfuehrer Thomas Wessel, William A. Martin, Margaret Seif

  reply	other threads:[~2007-05-25  9:45 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-23 13:19 Kenneth Zadeck
2007-05-24  1:29 ` Andrew_Pinski
2007-05-24  7:51   ` Paolo Bonzini
2007-05-24  8:12     ` Paolo Bonzini
2007-05-24 11:47 ` Bernd Schmidt
2007-05-24 20:02   ` Kenneth Zadeck
2007-05-25  9:51     ` Bernd Schmidt [this message]
2007-05-25 10:32       ` Paolo Bonzini
2007-05-25 13:00       ` Kenneth Zadeck
2007-05-28 20:55       ` Mark Mitchell
2007-05-29  1:16         ` Kenneth Zadeck
2007-05-29 15:58           ` Mark Mitchell
2007-05-24 21:09   ` Steven Bosscher
2007-05-24 22:23     ` Vladimir N. Makarov
2007-05-24 23:03       ` Steven Bosscher
2007-05-25 13:50         ` Vladimir N. Makarov
2007-05-25 17:15           ` Richard Sandiford
2007-05-25 18:21             ` Jan Hubicka
2007-05-25 20:13             ` Steven Bosscher
2007-05-25 20:11           ` Steven Bosscher
     [not found] <OF781E582C.3916D180-ON882572E5.00078CD6-882572E5.00082C95@LocalDomain>
2007-05-25  0:12 ` Andrew_Pinski
2007-05-25 19:04 Zack Weinberg
     [not found] <OF1BD0F1DA.E933CF3F-ON422572E9.006B2EF9-422572E9.006B85BF@de.ibm.com>
2007-05-29  1:05 ` Kenneth Zadeck
2007-05-30 13:11   ` Andreas Krebbel1
2007-05-30 14:17     ` Kenneth Zadeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4656AFFD.1040605@t-online.de \
    --to=bernds_cb1@t-online.de \
    --cc=Ulrich.Weigand@de.ibm.com \
    --cc=andrew_pinski@playstation.sony.com \
    --cc=belyshev@depni.sinp.msu.ru \
    --cc=bonzini@gnu.org \
    --cc=dberlin@dberlin.org \
    --cc=dje@watson.ibm.com \
    --cc=echristo@apple.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=iant@google.com \
    --cc=richard.earnshaw@arm.com \
    --cc=seongbae.park@gmail.com \
    --cc=stevenb.gcc@gmail.com \
    --cc=zadeck@naturalbridge.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).