public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Daniel Berlin <dan@cgsoftware.com>
To: Joe Buck <jbuck@synopsys.com>
Cc: dan@cgsoftware.com (Daniel Berlin),
	pfeifer@dbai.tuwien.ac dot@(Gerald Pfeifer),
	mark@codesourcery.com (Mark Mitchell),
	gcc@gcc.gnu.org (gcc@gcc.gnu.org)
Subject: Re: C++ compile-time regressions (was: GCC 3.0.1 Status Report)
Date: Tue, 24 Jul 2001 11:08:00 -0000	[thread overview]
Message-ID: <877kwygpyv.fsf@cgsoftware.com> (raw)
In-Reply-To: <200107241748.KAA08491@atrus.synopsys.com>

Joe Buck <jbuck@synopsys.com> writes:

>> Joe Buck <jbuck@synopsys.COM> writes:
>> 
>> >> I'm rather sure, though, that g++ is not generating efficient code for
>> >> complicated, nested STL and STL-like structures in the presence of
>> >> libstdc++-v3 (iterators are now classes instead of pointers).
>> > 
>> > But the classes have one member, so we should still be getting equivalent
>> > code, provided that we get ADDRESSOF working as well as it did in
>> > 2.95.x.
>> 
>> Not necessarily.
>> Alias analysis for classes says they can alias anything (It puts them
>> in set 0).
>> Alias analysis for pointers doesn't.
> 
> I'm very skeptical of this reasoning.
> 
> We're typically talking about code where, after inlining, we can see
> *exactly* what the pointer or reference refers to, because it gets set
> exactly once.


> 
> For the case where ADDRESSOF works, the class object should stay in a
> register, so there's no question of aliasing with something in memory.
> The problem is with making a decision to put an object in memory too
> early.
> 
> The iterator is a class, yes, but it has only one data member, which is
> a pointer.  There shouldn't be any impact on the aliasing analysis.

Yes yes, you are correct, i'm just cleaning up the mess something else
makes, in effect.

You (I would, but i'm having fun with new register allocator work,
and callgraph and loop stuff on trees right now. Maybe in a few more
months) should make a go at real scalar replacement of aggregates in
the ast-optimizer branch. 



-- 
"It's a good apartment because they allow pets.  I have a
Shetland pony named Nikkie.  Last summer Nikkie was involved in
a bizarre electrolysis accident.  All her hair was removed
except for her tail.  Now I rent her out to Hare Krishna family
picnics.
"-Steven Wright

  reply	other threads:[~2001-07-24 11:08 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-07-18 18:41 GCC 3.0.1 Status Report Mark Mitchell
2001-07-19  6:17 ` Joseph S. Myers
2001-07-20  3:45 ` Nathan Sidwell
2001-07-20 10:26 ` C++ compile-time regressions (was: GCC 3.0.1 Status Report) Gerald Pfeifer
2001-07-20 10:54   ` Nathan Sidwell
2001-07-20 16:41     ` Loren James Rittle
2001-07-23  5:17       ` Gerald Pfeifer
2001-07-23 11:56       ` Daniel Berlin
2001-07-23  2:51     ` Gerald Pfeifer
2001-07-20 10:55   ` Mark Mitchell
2001-07-20 14:22     ` Joe Buck
2001-07-20 16:28       ` Mark Mitchell
2001-07-20 21:05         ` Joe Buck
2001-07-23 10:44           ` Gerald Pfeifer
2001-07-23 11:12             ` Joe Buck
2001-07-23 11:57               ` Gerald Pfeifer
2001-07-23 13:30                 ` Joe Buck
2001-07-24  7:18                   ` Gerald Pfeifer
2001-07-24  7:55                     ` Daniel Berlin
2001-07-24 10:09                     ` Joe Buck
2001-07-24 10:19                       ` Gabriel Dos Reis
2001-07-24 10:44                         ` Joe Buck
2001-07-24 10:56                           ` Gabriel Dos Reis
2001-07-24 10:25                       ` Daniel Berlin
2001-07-24 10:49                         ` Joe Buck
2001-07-24 11:08                           ` Daniel Berlin [this message]
2001-07-23 14:36                 ` Mark Mitchell
2001-07-25  8:06                   ` Gerald Pfeifer
2001-08-02  0:20                     ` C++ compile-time regressions Gerald Pfeifer
2001-08-02  0:26                       ` Mark Mitchell
2001-08-02 11:00                       ` aoliva
2001-08-02 12:00                         ` Mark Mitchell
2001-08-02 12:26                           ` aoliva
2001-08-02 12:37                             ` Joe Buck
2001-08-02 12:41                             ` Mark Mitchell
2001-08-02 12:48                               ` Joe Buck
2001-08-02 13:53                       ` Daniel Berlin
2001-08-07  7:54                         ` Gerald Pfeifer
2001-07-23 11:54             ` C++ compile-time regressions (was: GCC 3.0.1 Status Report) Daniel Berlin
2001-07-23 12:42               ` Bernd Schmidt
2001-07-23 12:50               ` law
2001-07-23 13:15                 ` Daniel Berlin
2001-07-23 14:00                   ` law
2001-07-20 10:57   ` Daniel Berlin
2001-07-20 14:20   ` Joe Buck
2001-07-25 17:52 ` GCC 3.0.1 Status Report Marc Espie
2001-08-13 23:15   ` Mark Mitchell

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=877kwygpyv.fsf@cgsoftware.com \
    --to=dan@cgsoftware.com \
    --cc=jbuck@synopsys.com \
    --cc=pfeifer@dbai.tuwien.ac \
    /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).