public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "steven at gcc dot gnu dot org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug optimization/6585] Reduntant store/load instruction pairs on ix86
Date: Thu, 10 Jul 2003 20:09:00 -0000	[thread overview]
Message-ID: <20030710200916.30878.qmail@sources.redhat.com> (raw)
In-Reply-To: <20020506130600.6585.bruno@clisp.org>

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


steven at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|useless memory store        |Reduntant store/load
                   |instructions on x86         |instruction pairs on ix86


------- Additional Comments From steven at gcc dot gnu dot org  2003-07-10 20:09 -------
After looking into this a bit further and after consulting with Zdenek, I
believe we should blaim this bug on something in the register allocator.  But
then again the same problem exists with -fnew-ra, so maybe it's something in the
i386 backend (register sets???).  Maybe someone can try on a different 32bits
target and see if the problem exists there as well.

Anyway, before register allocation it
looks reasonable, but after global reg alloc we get this insns:

(insn 39 18 24 0 (set (mem:SI (plus:SI (reg/f:SI 7 esp)
                (const_int 4 [0x4])) [3 S4 A8])
        (reg:SI 2 ecx)) 39 {*movsi_1_nointernunit} (nil)
    (nil))
 
(note:HI 24 39 27 0 NOTE_INSN_FUNCTION_END)
 
(insn:HI 27 24 30 0 (set (reg/i:DI 0 rax [ <result> ])
        (mem:DI (reg/f:SI 7 esp) [3 S8 A8])) 60 {*movdi_2} (insn_list 18
(nil))
    (nil))

%ecx is put in the stack and subsequently loaded into %edx when the
result insn is expanded.  (After some black magic apparently %ecx is
replaced with %ebx post reload...)


  parent reply	other threads:[~2003-07-10 20:09 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20020506130600.6585.bruno@clisp.org>
2003-07-04 11:09 ` [Bug optimization/6585] useless memory store instructions on x86 steven at gcc dot gnu dot org
2003-07-10 20:09 ` steven at gcc dot gnu dot org [this message]
2003-08-23  1:38 ` [Bug optimization/6585] Redundant store/load instruction pairs on ix86 dhazeghi at yahoo dot com
2003-11-25  7:51 ` pinskia at gcc dot gnu dot org
2004-03-17  8:03 ` kazu at cs dot umass dot edu
2004-03-17 12:10 ` bruno at clisp dot org
2004-10-11  2:57 ` [Bug rtl-optimization/6585] " pinskia at gcc dot gnu dot org
2004-10-11 11:55 ` bruno at clisp dot org
2005-06-26 13:35 ` steven at gcc dot gnu dot org
2005-06-27 11:50 ` bruno at clisp 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=20030710200916.30878.qmail@sources.redhat.com \
    --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).