public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jan Hubicka <jh@suse.cz>
To: Richard Henderson <rth@redhat.com>,
	Ulrich Weigand <Ulrich.Weigand@de.ibm.com>,
	Jan Hubicka <jh@suse.cz>,
	gcc@gcc.gnu.org, aj@suse.de
Subject: Re: Profiling on S390
Date: Wed, 23 Apr 2003 10:59:00 -0000	[thread overview]
Message-ID: <20030423074537.GG15464@kam.mff.cuni.cz> (raw)
In-Reply-To: <20030422211923.GE21400@redhat.com>

> On Tue, Apr 22, 2003 at 10:43:37PM +0200, Ulrich Weigand wrote:
> > >It looks very unsafe for me - GCC assumes at many places that CC is not
> > >live accross boundaries.
> > 
> > This is annoying.  Is there any fundamental reason for this restriction?
> 
> No, and it's not actually true.
> 
> What is true is that you can't generally move the thing around
> if most of your instructions clobber the flags.  So e.g. for x86
> this would be a Really Bad Idea, but for Sparc it can work out.

My impression is that s390 is x86-like when dealing with flags.
> 
> It's also true that some of the simplifiers (cse, combine) won't
> DTRT unless the compare and use are in the same block.
> 
> > Can't the profiler use special code to save/restore CC?
> 
> It could, but if you don't have add/load/store insns that don't
> clobber CC, then you'll get in trouble with reload too.  Of
We can conclude to use some named patterns.  For instance
"save_state" "restore_state" to fold profiler code in.  Does this sound
plausible?

Honza
> course, that brings up the issue of getting the right pattern
> emitted from the profiler.  Could be done with a special named
> add pattern, but that seems like a lot of overhead for very
> little return.
> 
> This is all about a movstrsi or cmpstrsi pattern?  Perhaps it
> would be better to keep this as one unit until after reload?
> 
> 
> r~

  reply	other threads:[~2003-04-23  7:45 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-22 22:34 Ulrich Weigand
2003-04-22 23:13 ` Richard Henderson
2003-04-23 10:59   ` Jan Hubicka [this message]
2003-04-23 18:26     ` Richard Henderson
2003-04-23 19:18       ` Jan Hubicka
2003-04-23 19:56         ` Richard Henderson
2003-04-23 21:29           ` Jan Hubicka
2003-04-23 22:33             ` Richard Henderson
2003-04-25 17:01               ` Jan Hubicka
  -- strict thread matches above, loose matches on Subject: below --
2003-04-23 21:50 Ulrich Weigand
2003-04-23 22:08 ` Richard Henderson
2003-04-23 18:18 Ulrich Weigand
2003-04-23 12:19 Ulrich Weigand
2003-04-18 12:11 Jan Hubicka

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=20030423074537.GG15464@kam.mff.cuni.cz \
    --to=jh@suse.cz \
    --cc=Ulrich.Weigand@de.ibm.com \
    --cc=aj@suse.de \
    --cc=gcc@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).