public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "law at redhat dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug tree-optimization/18576] [3.4/4.0 Regression] missing jump threading because of type changes
Date: Thu, 25 Nov 2004 00:22:00 -0000	[thread overview]
Message-ID: <20041125002219.28168.qmail@sourceware.org> (raw)
In-Reply-To: <20041120062915.18576.pinskia@gcc.gnu.org>


------- Additional Comments From law at redhat dot com  2004-11-25 00:22 -------
Subject: Re:  [3.4/4.0 Regression] missing
	jump threading because of type changes

On Wed, 2004-11-24 at 22:38 +0000, steven at gcc dot gnu dot org wrote:
> ------- Additional Comments From steven at gcc dot gnu dot org  2004-11-24 22:38 -------
> I don't think this is "minor".  We have many places where we have 
> predicates returning "bool", and this PR suggests that we can never 
> thread on their results because of casts. 
>  
> Adding Jeff, this may be a new prey for him after he's done with 
> Bug 15524 ;-) 
There are a number of ways we can attack this.  I don't know if any are
appropriate for 4.0 though.

One approach is to go with the phi optimization rewrite which allows it
to work with > 2 predecessors to the PHI block.

Another would be to look at Kazu's work which (IIRC) tried to identify
typecasts which were fed by PHIs with all constant arguments.  It then
removed the typecast and original PHI and installed a new PHI with a
result of the proper type.

A third would be to revamp the jump threading selection code.   In the
early days of the jump threading code we couldn't handle SSA graph
updates if we threaded through a block with side effects.  The new
SSA graph update code handles that case, so the selection code shouldn't
need to be so conservative anymore.

I think ultimately we'll want to do all three.  The question in my mind
is which (if any) are suitable for gcc 4.0.

jeff




-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18576


  parent reply	other threads:[~2004-11-25  0:22 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-20  6:29 [Bug tree-optimization/18576] New: " pinskia at gcc dot gnu dot org
2004-11-20  6:37 ` [Bug tree-optimization/18576] " pinskia at gcc dot gnu dot org
2004-11-20  6:43 ` pinskia at gcc dot gnu dot org
2004-11-20 16:44 ` steven at gcc dot gnu dot org
2004-11-20 17:01 ` steven at gcc dot gnu dot org
2004-11-20 17:11 ` steven at gcc dot gnu dot org
2004-11-20 17:11 ` pinskia at gcc dot gnu dot org
2004-11-20 17:57 ` [Bug tree-optimization/18576] [3.4/4.0 Regression] " pinskia at gcc dot gnu dot org
2004-11-24 22:38 ` steven at gcc dot gnu dot org
2004-11-25  0:22 ` law at redhat dot com [this message]
2004-12-12 17:26 ` kazu at cs dot umass dot edu
2004-12-12 17:42 ` kazu at cs dot umass dot edu
2004-12-13 16:33 ` law at redhat dot com
2004-12-13 17:13 ` kazu at cs dot umass dot edu
2004-12-13 17:28 ` law at redhat dot com
2004-12-22 14:08 ` kazu at cs dot umass dot edu
2005-01-06 14:43 ` tobi at gcc dot gnu dot org
2005-03-22 19:26 ` pinskia at gcc dot gnu dot org
2005-05-19 17:38 ` mmitchel at gcc dot gnu dot org
2005-07-22 21:12 ` pinskia at gcc dot gnu dot org
2005-09-27 16:17 ` mmitchel at gcc dot gnu dot org
     [not found] <bug-18576-6528@http.gcc.gnu.org/bugzilla/>
2005-11-04 15:37 ` pinskia at gcc dot gnu dot org

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=20041125002219.28168.qmail@sourceware.org \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /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).