Tree inlining for the C front end (part 3 of 3)

law@redhat.com law@redhat.com
Wed Sep 26 15:29:00 GMT 2001


  In message < orvgi75s3p.fsf@localhost.localdomain >you write:
  > > Had performance testing been done, it would have been noticed that it
  > > made 0% difference.
  > 
  > I guess performance testing was done at the time store motion was
  > implemented.  I suspect it ended up broken because of countless
  > merges, possibly with errors, before it was contributed.  This is
  > exactly what I'm trying to avoid by contributing this chunk of code as
  > early as possible.
Also note that store motion was initially implemented to deal with a
specific class of store motions -- if you did anything outside the
very limited set of things it knew about no motions would be performed.

The idea (of course) was to get the base algorithms working correctly,
then tie in the aliasing code to make them more effective.

For various reasons I think it's highly unlikely Daniel had the
original store motion code as it was originally done on a customer
specific branch inside Cygnus's repository.  ANd yes, the various
merges, including the merge into the head of Cygnus's repository
were rather brutal.

jeff



More information about the Gcc-patches mailing list