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 target/52897] gcc 4.7.0 generates worse code than gcc 3.4.6 for m68000
Date: Fri, 16 Jan 2015 19:44:00 -0000	[thread overview]
Message-ID: <bug-52897-4-O1r4kOxMf8@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-52897-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52897

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P5
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-01-16
                 CC|                            |law at redhat dot com
     Ever confirmed|0                           |1

--- Comment #7 from Jeffrey A. Law <law at redhat dot com> ---
It would be best to describe the m68k port as in maintenance mode only.  As
Mikael noted, it's certainly not a priority anymore, but we do make an effort
to fix correctness issues, particularly those which are regressions relative to
prior releases.

Code generation improvements are significantly lower in terms of priority. 
However, if you're able to show the same code regressing from a performance
standpoint on another more common platform (say m68k, aarch64, power), then
there's a better chance someone will devote some time to the issue since it's
likely a target independent missed optimization.

The first thing I notice when I look at this bug is that as we come out of the
SSA optimizers we've split the memory references.  That isn't inherently wrong
as we have RTL optimizers that ought to be able to put them back together.  
But that doesn't happen because the key memory objects are declared volatile.

If I remove the volatile designation I get dramatically better code because we
end up doing memory->memory moves rather than load into a register then a store
from the register.

I suspect this particular issue is only going to show up on targets where we
have memory-to-memory moves which is relatively rare.


      parent reply	other threads:[~2015-01-16 19:44 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-07  8:36 [Bug c/52897] New: " fdarkangel at gmail dot com
2012-04-07  8:37 ` [Bug c/52897] " fdarkangel at gmail dot com
2012-04-07 10:33 ` mikpe at it dot uu.se
2013-12-18  4:35 ` [Bug target/52897] " fdarkangel at gmail dot com
2014-09-12  2:34 ` fdarkangel at gmail dot com
2014-09-13  9:57 ` mikpelinux at gmail dot com
2014-09-13 16:10 ` fdarkangel at gmail dot com
2015-01-16 19:44 ` law at redhat dot com [this message]

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=bug-52897-4-O1r4kOxMf8@http.gcc.gnu.org/bugzilla/ \
    --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).