public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jan Hubicka <jh@suse.cz>
To: Daniel Berlin <dan@cgsoftware.com>
Cc: Jan Hubicka <jh@suse.cz>, Richard Henderson <rth@redhat.com>,
	gcc@gcc.gnu.org, m.hayes@elec.canterbury.ac.nz,
	matzmich@cs.tu-berlin.de
Subject: Re: df.c and partial writes/REG_EQUAL notes
Date: Thu, 27 Sep 2001 07:58:00 -0000	[thread overview]
Message-ID: <20010927165837.C23745@atrey.karlin.mff.cuni.cz> (raw)
In-Reply-To: <87elosbsx9.fsf@cgsoftware.com>

> Don't bother, i already did it.
> You'll note that reload makes too many bitmaps, and the 16-24 bytes
> more overhead + the empty array pointers that the ebitmap takes causes
> 100-150 meg more memory.
> Just another point that we shouldn't up and replace them wholesale,
> rather do it incrementally.
> 
> I have resolved this problem by reworking the internals to
> not need to store empty words at all (ie not even pointers to them).

How did you accomplished that?
Is that contained in the updated ebitmap patch?
> 
> I also replaced bitmaps with them already.  The new internal storage
> mechanism requires changes to sbitmaps so you can properly
> and/or/copy/etc bitmaps of differing bit sizes.
> I.E. it'll do the right thing, as long as the destination is big
> enough to hold the right result. sbitmap_a_or_b (350 bit sbitmap, 25
> bit sbitmap, 200 bit sbitmap) will work properly.
> It would and/or/etc random memory before.
> 
> However, experience in doing this just helps prove my point. I tried
> doing it all at once, and it made finding the real cause of the bugs
> impossible.

Thats understandable - it happends in gcc commonly :(
> 
> > > 
> >> This was your "review".
> > Please lets work together to get gcc better, not fight.
> Who is fighting?
> I put review in quotes because i do not believe he reviewed the patch
> itself, as i stated in my message to Gerald.
> 
> I should also note that there was a bug in ebitmap.h in
> EXECUTE_IF_SET_IN_EBITMAP that a meaningful review of the patch would
> have caught. 
> 
> >
> > Honza
> > > 
> >> >
> >> > r~
> >> 
> >> -- 
> >> "When I was five years old I was on a merry go round.  There was
> >> a gunshot nearby.  The horses stampeded.  There I was running
> >> down the street on a purple wooden horse.
> >> "-Steven Wright
> 
> -- 
> "Last time I went skiing, I had to get up at 5:00 in the morning.
> I knew I couldn't do that, so I slept with my skis on.  My ride
> came at 5:30 in the morning, couldn't wake me up so he carried
> me out of the house, put my skis on the roof rack of the car,
> and drove to the mountain.  Seventeen miles later, I woke up out
> of this incredibly bizarre dream that I was skydiving
> horizontally.  I'm sure this has happened to you.
> "-Steven Wright

  reply	other threads:[~2001-09-27  7:58 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-25  7:16 Jan Hubicka
2001-09-25  7:55 ` Daniel Berlin
2001-09-25  7:59   ` Jan Hubicka
2001-09-25  8:33     ` Daniel Berlin
2001-09-25  8:35       ` Jan Hubicka
2001-09-26 13:03       ` Richard Henderson
2001-09-27  6:29         ` Daniel Berlin
2001-09-27  7:13           ` Gerald Pfeifer
2001-09-27  7:28             ` Daniel Berlin
2001-09-27  7:47               ` David Edelsohn
2001-09-27  7:27           ` Jan Hubicka
2001-09-27  7:38             ` Daniel Berlin
2001-09-27  7:58               ` Jan Hubicka [this message]
2001-09-27  9:40                 ` Daniel Berlin
2001-09-27 10:27           ` Richard Henderson
2001-09-27 11:21             ` Daniel Berlin
2001-10-01 17:07             ` Mark Mitchell
2001-09-27 20:12           ` law
2001-09-28 10:49     ` Michael Matz
2001-09-29  6:38       ` Jan Hubicka
2001-10-24 12:35 ` law
2001-10-24 13:00   ` Richard Henderson
2001-10-24 13:27   ` Michael Matz
2001-10-25 11:20     ` law
2001-10-25  6:23   ` Jan Hubicka
2001-10-25  6:36     ` Daniel Berlin
2001-09-27  8:52 Richard Kenner
2001-09-27 10:20 ` Jan Hubicka
2001-09-27 15:11   ` Daniel Berlin
2001-09-27 11:41 ` Daniel Berlin
2001-09-27 11:48 Richard Kenner
2001-09-27 14:16 mike stump

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=20010927165837.C23745@atrey.karlin.mff.cuni.cz \
    --to=jh@suse.cz \
    --cc=dan@cgsoftware.com \
    --cc=gcc@gcc.gnu.org \
    --cc=m.hayes@elec.canterbury.ac.nz \
    --cc=matzmich@cs.tu-berlin.de \
    --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).