public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "pinskia at gcc dot gnu dot org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/26525] missed opportunity to use value in fp reg
Date: Thu, 02 Mar 2006 03:07:00 -0000	[thread overview]
Message-ID: <20060302030743.1369.qmail@sourceware.org> (raw)
In-Reply-To: <bug-26525-1057@http.gcc.gnu.org/bugzilla/>



------- Comment #1 from pinskia at gcc dot gnu dot org  2006-03-02 03:07 -------
This is definitely a RA issue.
(insn:HI 12 65 15 2 (set (reg/v:DI 121 [ hi ])
        (fix:DI (reg/v:DF 123 [ xh ]))) 265 {fix_truncdfdi2}
(insn_list:REG_DEP_TRUE 6 (nil))
    (nil))

(insn:HI 15 12 16 2 (set (reg:CC 125)
        (compare:CC (reg/v:DI 121 [ hi ])
            (reg:DI 124))) 412 {*cmpdi_internal1} (insn_list:REG_DEP_TRUE 65
(insn_list:REG_DEP_TRUE 12 (nil)))
    (expr_list:REG_DEAD (reg:DI 124)
        (expr_list:REG_EQUAL (compare:CC (reg/v:DI 121 [ hi ])
                (const_int 9223372036854775807 [0x7fffffffffffffff]))
            (nil))))
...
(insn:HI 36 34 37 4 (set (reg:DF 132)
        (float:DF (reg/v:DI 121 [ hi ]))) 262 {floatdidf2} (nil)
    (expr_list:REG_DEAD (reg/v:DI 121 [ hi ])
        (nil)))

The problem is that we need to put 121 into an integer register for the compare
but also in a fp register for the int-to-float.

Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |missed-optimization, ra
   Last reconfirmed|0000-00-00 00:00:00         |2006-03-02 03:07:41
               date|                            |


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


  reply	other threads:[~2006-03-02  3:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-02  2:40 [Bug target/26525] New: " amodra at bigpond dot net dot au
2006-03-02  3:07 ` pinskia at gcc dot gnu dot org [this message]
2006-06-23  1:41 ` [Bug target/26525] " pinskia at gcc dot gnu dot org
2007-10-23 17:13 ` pinskia at gcc dot gnu dot org
2008-09-14  3:28 ` pinskia at gcc dot gnu dot org
2009-05-14  3:19 ` bje 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=20060302030743.1369.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).