public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Mike Stump <mrs@apple.com>
To: Dave Korn <dave.korn@artimi.com>
Cc: "'Richard Earnshaw'" <rearnsha@gcc.gnu.org>,
	        "'Ian Lance Taylor'" <ian@airs.com>,
	gcc@gcc.gnu.org
Subject: Re: Why doesn't combine like volatiles? (volatile_ok again, sorry!)
Date: Mon, 28 Nov 2005 18:14:00 -0000	[thread overview]
Message-ID: <DC23BBCB-C969-4E93-8ADA-8FD8F244952B@apple.com> (raw)
In-Reply-To: <SERRANODll7fnp3eEHJ000001cb@SERRANO.CAM.ARTIMI.COM>

On Nov 28, 2005, at 9:29 AM, Dave Korn wrote:
>   BTW, I never did manage to find the patches you referred to in  
> your postings
> from summer 2000.  Googling for "mike stump volatile_ok" just kept  
> on finding
> me the post where you were advising someone to find your patches by  
> searching
> for your name and volatile_ok.  Kinda recursive, that.... do you  
> still have a
> pointer to them?

As background for others,

http://gcc.gnu.org/ml/gcc-bugs/1999-12n/msg00801.html

has an example of why one would want to do this.

Anyway, not rocket science:

Doing diffs in .:
--- ./recog.c.~1~       2005-10-28 10:40:18.000000000 -0700
+++ ./recog.c   2005-11-28 10:10:31.000000000 -0800
@@ -956,9 +956,6 @@ general_operand (rtx op, enum machine_mo
      {
        rtx y = XEXP (op, 0);

-      if (! volatile_ok && MEM_VOLATILE_P (op))
-       return 0;
-
        /* Use the mem's mode, since it will be reloaded thus.  */
        if (memory_address_p (GET_MODE (op), y))
         return 1;
--------------

Good for testing, but a real patch would have to:

http://gcc.gnu.org/ml/gcc/2001-11/msg00398.html

  reply	other threads:[~2005-11-28 18:14 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-21 18:12 Dave Korn
2005-11-21 21:46 ` Ian Lance Taylor
2005-11-22 10:45   ` Richard Earnshaw
2005-11-22 15:33     ` Dave Korn
2005-11-22 15:52       ` Richard Earnshaw
2005-11-22 16:10         ` Robert Dewar
2005-11-27  3:18           ` Mike Stump
2005-11-27  3:14         ` Mike Stump
2005-11-28 11:01           ` Richard Earnshaw
2005-11-28 11:13             ` Robert Dewar
2005-11-28 13:09               ` Paolo Bonzini
2005-11-28 21:05                 ` Laurent GUERBY
2005-11-28 22:26                   ` Robert Dewar
2005-11-28 17:14               ` Mike Stump
2005-11-28 17:19                 ` Andrew Pinski
2005-11-28 17:22                   ` Mike Stump
2005-11-28 17:35                   ` Mike Stump
2005-11-28 17:41                     ` Andrew Pinski
2005-11-28 17:53                       ` Mike Stump
2005-11-28 18:08                         ` Joe Buck
2005-11-28 18:28                           ` Mike Stump
2005-11-28 19:12                         ` Robert Dewar
2005-11-28 19:56                           ` Mike Stump
2005-11-28 20:01                             ` Robert Dewar
2005-11-28 20:18                               ` Gabriel Dos Reis
2005-11-28 20:21                                 ` Robert Dewar
2005-11-28 19:05                 ` Robert Dewar
2005-11-28 19:53                   ` Mike Stump
2005-11-28 17:11             ` Mike Stump
2005-11-28 17:29               ` Dave Korn
2005-11-28 18:14                 ` Mike Stump [this message]
2005-11-23 19:54   ` Accidentally on the list Eric J. Goforth
2005-11-23 19:56     ` Gerald Pfeifer
2005-11-22 16:16 Why doesn't combine like volatiles? (volatile_ok again, sorry!) Richard Kenner
2005-11-22 18:38 ` Robert Dewar
2005-11-22 19:02   ` Dave Korn
2005-11-22 19:10     ` Robert Dewar
2005-11-22 20:39       ` Dave Korn
2005-11-23 11:27       ` Richard Earnshaw
2005-11-23 11:41         ` Robert Dewar
2005-11-23 12:04           ` Richard Earnshaw
2005-11-23 12:15             ` Robert Dewar
2005-11-28 17:28 Richard Kenner
2005-11-28 17:44 ` Mike Stump
2005-11-28 18:50 Richard Kenner
2005-11-28 20:25 ` Mike Stump
2005-11-28 20:40   ` Andrew Pinski
2005-11-28 21:55     ` Mike Stump
2005-11-28 22:14       ` Andrew Pinski

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=DC23BBCB-C969-4E93-8ADA-8FD8F244952B@apple.com \
    --to=mrs@apple.com \
    --cc=dave.korn@artimi.com \
    --cc=gcc@gcc.gnu.org \
    --cc=ian@airs.com \
    --cc=rearnsha@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).