public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Jakub Jelinek <jakub@redhat.com>, Andrew Haley <aph@redhat.com>
Cc: Alexandre Oliva <aoliva@redhat.com>,
	Richard Henderson <rth@redhat.com>,
	     gcc-patches@gcc.gnu.org,
	classpath-patches ml <classpath-patches@gnu.org>,
	     Java Patch List <java-patches@gcc.gnu.org>
Subject: Re: Split gnu/javax/swing/text/html/parser/HTML_401F.java
Date: Sun, 12 Jul 2009 20:27:00 -0000	[thread overview]
Message-ID: <alpine.LSU.1.99.0907122225070.8877@acrux.dbai.tuwien.ac.at> (raw)
In-Reply-To: <4A4B5883.4050905@redhat.com>

On Wed, 1 Jul 2009, Andrew Haley wrote:
>> I haven't studied how exactly is --enable-java-maintainer-mode
>> compiling the classes; if I just gcj -C HTML_401F.java on
>> Fedora 11 (GCC 4.4.0, ecj 3.4.2), the compile time with patched
>> VTA is only 4:53 with 1.5GB top memory usage, if I patch HTML_401F.java
>> with the following patch, it compiles within 0:55 and maxes at 250MB.

That's quite a nice improvement.  HTML_401F.java has been causing 
troubles for many years, and splitting it really helps, for example
building on (virtual) machines with not so much main memory or in
limited settings where there is a process limit for 512MB.

> It's not an ABI change.  This patch is OK iff accompanied by a
> comment in the code that explains the problem.

I believe the patch has not made it into GCC Subversion yet.  Are
the two of you still planning to apply it?

Gerald

  parent reply	other threads:[~2009-07-12 20:27 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20090630124346.GX4462@tyan-ft48-01.lab.bos.redhat.com>
     [not found] ` <alpine.LSU.1.99.0906301619520.29901@acrux.dbai.tuwien.ac.at>
     [not found]   ` <4A4A2623.9010609@redhat.com>
     [not found]     ` <20090630152911.GA4462@tyan-ft48-01.lab.bos.redhat.com>
2009-07-01 12:37       ` Split gnu/javax/swing/text/html/parser/HTML_401F.java (Was [vta] Add chains from referenced VALUEs to DVs that reference them) Andrew Haley
2009-07-01 19:21         ` Jakub Jelinek
2009-07-02  8:15           ` Andrew Haley
2009-07-12 20:27         ` Gerald Pfeifer [this message]
2009-07-12 20:36           ` Split gnu/javax/swing/text/html/parser/HTML_401F.java Jakub Jelinek
2009-07-12 21:00             ` Gerald Pfeifer
2009-07-14 16:28             ` [cp-patches] " Audrius Meskauskas
2009-07-27 22:55               ` Andrew John Hughes
2009-07-29  7:06                 ` Andrew Haley
2009-08-02 21:44                   ` Audrius Meskauskas
2009-07-13  9:31           ` Andrew Haley

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=alpine.LSU.1.99.0907122225070.8877@acrux.dbai.tuwien.ac.at \
    --to=gerald@pfeifer.com \
    --cc=aoliva@redhat.com \
    --cc=aph@redhat.com \
    --cc=classpath-patches@gnu.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@redhat.com \
    --cc=java-patches@gcc.gnu.org \
    --cc=rth@redhat.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).