public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* Re: Some Haifa scheduler bugs
@ 1997-08-19 17:54 Jeffrey A Law
  1997-08-19 17:54 ` egcs: A new compiler project to merge the existing GCC forks Toon Moene
  1997-08-19 17:54 ` egcs: A new compiler project to merge the existing GCC forks (fwd) Dave Love
  0 siblings, 2 replies; 11+ messages in thread
From: Jeffrey A Law @ 1997-08-19 17:54 UTC (permalink / raw)
  To: egcs

  In message <Pine.SOL.3.90.970819092828.291B-100000@starsky.informatik.rwth-aa
chen.de>you write:
  > I've run c-torture on the egcs snapshot, using the Haifa scheduler with
  > all flags turned on. My system is an i586-linux one. Here's a patch to fix
  > some of the failures.
Thanks.

Just some comments:

  * Get a copyright assignment + disclaimer signed and sent to
  the FSF as soon as possible.  Until that time we can't take any
  of your patches and include them without rewriting them first.

  * When submitting patches, send separate patches for bugs from
  random cleanups.  The cleanups are greatly appreciated, especially
  for haifa.

  * When submitting bugfix patches, please submit a testcase, or
  refer us to a c-torture testcase and the options needed to expose
  the bug.


Specific questions/comments:

In move_insn, is there some reason why you can't call reemit_notes
during the loop on SCHED_GROUP_P insns?

ie, does this work instead?  Seems cleaner than using another loop
if it works.

{
  rtx new_last = insn;

  while (SCHED_GROUP_P (insn))
    {
      rtx prev = PREV_INSN (insn);
      move_insn1 (insn, last);
      reemit_notes (insn, insn);
      insn = prev;
    }

  move_insn1 (insn, last);
  return reemit_notes (new_last, new_last);
}

[ Of course if you had referred us to a testcase, we could check
  this ourselves.... ]

Jeff

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: egcs: A new compiler project to merge the existing GCC forks (fwd)
@ 1997-08-19 17:18 Joern Rennecke
  0 siblings, 0 replies; 11+ messages in thread
From: Joern Rennecke @ 1997-08-19 17:18 UTC (permalink / raw)
  To: egcs

>   In message <Pine.GSO.3.96.970819004905.4653A-100000@drabble>you write:
>   > can old scheduler be the source of the problem?
> Could be -- I don't think gcc-2.7* scheduled instructions on the
> x86 machines at all.
> 
> So, one interesting test would be to run the benchmark with "-O2",
> then again with "-O2 -fno-schedule-insns -fno-schedule-insns2".
> 
> That would tell us if we need to focus on the scheduler or not.

And if you look for the best performance right now, I suggest to try
-O2 -fno-schedule-insns .  Scheduling after reload can't hurt register
allocation, and it might do some good.  OTOH, it can hurt when it
disables peepholes.  It's a bity we don't have any actual peephole
optimization pass - combine.c works before reload and is limited in the
number and kind of insns it can combine, and the peepholes used by final
don't allow re-iteration and are not designed to recognize insns sequences
with some unrelated insns in-between.

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: egcs: A new compiler project to merge the existing GCC forks (fwd)
@ 1997-08-19 13:19 H.J. Lu
  0 siblings, 0 replies; 11+ messages in thread
From: H.J. Lu @ 1997-08-19 13:19 UTC (permalink / raw)
  To: egcs

> 
> 
> HJ, can you work with this person to find out _why_ performance
> is suffering?
> 

I am still working on prototyping. But if noone is looking at it,
I will take a look.


H.J.

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: Reload patch to improve 386 code
@ 1997-08-19  8:50 Jakub Jelinek
  1997-08-19  9:47 ` egcs: A new compiler project to merge the existing GCC forks (fwd) Dave Love
  0 siblings, 1 reply; 11+ messages in thread
From: Jakub Jelinek @ 1997-08-19  8:50 UTC (permalink / raw)
  To: egcs

> 
>    Date: Mon, 18 Aug 1997 11:17:38 -0400 (EDT)
>    From: meissner@cygnus.com
> 
>    I think it is an horrible kludge that reload is run as a pass after
>    global-alloc, and that it forces reload registers not to be used
>    for any other purpose (which is murder on the x86 with each
>    register being special purpose in some way).
> 
> Before this leaves my head, I wanted to point something out which
> you've reminded me of.  When the scheduler (this applies to both the
> original and Haifa versions equally) becomes aggressive, it produces a
> large number of reloads in certain situations.  Reloads which would
> not have happened if scheduling did not take place.  This happens
> especially if register pressure is high already.  I noticed this
> particularly on RISC platforms, seems in this case the more registers
> available the worse things became when the register usage was
> saturated.

I thought about a quick solution, which would be during global-alloc, if it
finds out that the number of hard registers is exceeded, it could try to
undo some short pseudo setup RTL sequence merges and move them to the place
of the actual use, if the pseudo being set up is a constant and computable
in small number of instructions not involving memory loads.
Like that, we could rid of the following horror on sparc64:

	sethi %hi(var1), %r1
	stx %r1, [%sp + NN]
	...
	ldx [%sp + NN], %r1
	or %r1, %lo(var1), %r1
	stx %r1, [%sp + NN]
	...
some loop:
	...
	ldx [%sp + NN], %r1
	ldx [%r1], %r1
	...

and could have:

some loop:
	...
	sethi %hi(var1), %r1
	ldx [%r1 + %lo(var1)], %r1
	...

instead...

Cheers,
    Jakub
___________________________________________________________________
Jakub Jelinek | jj@sunsite.mff.cuni.cz | http://sunsite.mff.cuni.cz
Administrator of SunSITE Czech Republic, MFF, Charles University
___________________________________________________________________
Ultralinux - first 64bit OS to take fool power from the UltraSparc
Linux version 2.0.30 on a sparc machine (291.64 BogoMips).
___________________________________________________________________

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: egcs: A new compiler project to merge the existing GCC forks (fwd)
@ 1997-08-19  7:36 Robert Wilhelm
  0 siblings, 0 replies; 11+ messages in thread
From: Robert Wilhelm @ 1997-08-19  7:36 UTC (permalink / raw)
  To: egcs

> 
> Can someone point me the location of mdbench?
>

http://www.sissa.it/furio/Mdbnch/info.html

Robert

^ permalink raw reply	[flat|nested] 11+ messages in thread
* egcs: A new compiler project to merge the existing GCC forks (fwd)
@ 1997-08-19  3:52 H.J. Lu
  1997-08-19  4:27 ` Jeffrey A Law
                   ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: H.J. Lu @ 1997-08-19  3:52 UTC (permalink / raw)
  To: egcs

Forwarded message:
Date: Tue, 19 Aug 1997 11:09:07 +0900
From: Arno PAHLER <paehler@atlas.rc.m-kagaku.co.jp>
Message-Id: <199708190209.LAA04886@atlas.rc.m-kagaku.co.jp>
To: "H.J. Lu" <hjl@lucon.org>
In-reply-to: "H.J. Lu"'s message of Sun, 17 Aug 1997 09:12:40 -0700
Subject: egcs: A new compiler project to merge the existing GCC forks


I downloaded this a few days ago - compiles and runs without any
problems on a PentiumPro Linux 2.0.30 (Redhat 4.2) system - but:

execution speed (floating point) of a test case (mdbench) compiled
with f2c+gcc is about 10% slower than using gcc 2.7.2.1 - it is
about the same or very slighly faster than g77 0.5.19.1 when using
g77 0.5.21 - when using single precision both f2c+gcc and g77 are
about 10-25% slower than their gcc 2.7.2.1/g77 0.5.19.1 counter-
parts.

I had hoped that performance would improve rather than get worse -
is it so hard to optimize for x86? - I am back right now to the old
stuff, unless I get to hear a convincing reason why to switch.


Arno


-- 
H.J. Lu (hjl@gnu.ai.mit.edu)

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~1997-08-19 17:54 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1997-08-19 17:54 Some Haifa scheduler bugs Jeffrey A Law
1997-08-19 17:54 ` egcs: A new compiler project to merge the existing GCC forks Toon Moene
1997-08-19 17:54 ` egcs: A new compiler project to merge the existing GCC forks (fwd) Dave Love
  -- strict thread matches above, loose matches on Subject: below --
1997-08-19 17:18 Joern Rennecke
1997-08-19 13:19 H.J. Lu
1997-08-19  8:50 Reload patch to improve 386 code Jakub Jelinek
1997-08-19  9:47 ` egcs: A new compiler project to merge the existing GCC forks (fwd) Dave Love
1997-08-19  7:36 Robert Wilhelm
1997-08-19  3:52 H.J. Lu
1997-08-19  4:27 ` Jeffrey A Law
1997-08-19  5:08 ` Oleg Krivosheev
1997-08-19  6:01 ` Jeffrey A Law

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).