From: Corey Minyard <minyard@acm.org>
To: Richard Henderson <rth@redhat.com>
Cc: gcc@gcc.gnu.org
Subject: Re: Loop optimization bug with Ada front end on PPC (and probably Alpha)
Date: Mon, 19 Nov 2001 07:58:00 -0000 [thread overview]
Message-ID: <3C040513.60803@acm.org> (raw)
In-Reply-To: <20011127125755.C30084@redhat.com>
Richard Henderson wrote:
>On Tue, Nov 27, 2001 at 01:52:41PM -0600, Corey Minyard wrote:
>
>>Unforunately, this causes my original problem to come back. I'm nervous
>>about this patch, too, I don't understand it at all.
>>
>
>This original problem is the Ada test case you forwarded,
>or something else?
>
yes, the original Ada testcase.
>>I tried moving this test to doloop_modify_runtime in doloop.c and adding
>>the increment to the "diff" rtx in this situation, and it works in my
>>simple testcase and didn't cause the problem in the regression.
>>
>
>Hum. Perhaps you're right -- putting this change in doloop
>will limit liability, so to speak. I'll move my change.
>
Are you moving back to the original loop test? I think the original
test you had didn't work because it wasn't the doloop case, but if you
put it in the doloop code it should work properly.
Bummer, my bootstrap just segv-ed. Oh well, a little more work, some
variable was probably NULL.
-Corey
WARNING: multiple messages have this Message-ID
From: Corey Minyard <minyard@acm.org>
To: Richard Henderson <rth@redhat.com>
Cc: gcc@gcc.gnu.org
Subject: Re: Loop optimization bug with Ada front end on PPC (and probably Alpha)
Date: Tue, 27 Nov 2001 13:25:00 -0000 [thread overview]
Message-ID: <3C040513.60803@acm.org> (raw)
Message-ID: <20011127132500.meM3w6ZcBUruIXRCW2yOY1c-yQudIEeCEJvZtpd7W54@z> (raw)
In-Reply-To: <20011127125755.C30084@redhat.com>
Richard Henderson wrote:
>On Tue, Nov 27, 2001 at 01:52:41PM -0600, Corey Minyard wrote:
>
>>Unforunately, this causes my original problem to come back. I'm nervous
>>about this patch, too, I don't understand it at all.
>>
>
>This original problem is the Ada test case you forwarded,
>or something else?
>
yes, the original Ada testcase.
>>I tried moving this test to doloop_modify_runtime in doloop.c and adding
>>the increment to the "diff" rtx in this situation, and it works in my
>>simple testcase and didn't cause the problem in the regression.
>>
>
>Hum. Perhaps you're right -- putting this change in doloop
>will limit liability, so to speak. I'll move my change.
>
Are you moving back to the original loop test? I think the original
test you had didn't work because it wasn't the doloop case, but if you
put it in the doloop code it should work properly.
Bummer, my bootstrap just segv-ed. Oh well, a little more work, some
variable was probably NULL.
-Corey
next prev parent reply other threads:[~2001-11-27 21:25 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-11-13 5:20 Corey Minyard
2001-11-13 6:05 ` David Edelsohn
2001-11-14 8:05 ` Corey Minyard
2001-11-13 8:02 ` guerby
2001-11-13 9:48 ` Corey Minyard
2001-11-16 22:19 ` David Edelsohn
2001-11-26 7:39 ` David Edelsohn
2001-11-15 0:48 ` Richard Henderson
2001-11-15 14:19 ` Andreas Schwab
2001-11-15 16:20 ` Richard Henderson
2001-11-25 12:23 ` Richard Henderson
2001-11-18 9:18 ` Richard Henderson
2001-11-27 0:02 ` Richard Henderson
2001-11-25 8:57 ` Andreas Schwab
2001-11-15 18:02 ` Corey Minyard
2001-11-15 19:37 ` Richard Henderson
2001-11-25 15:52 ` Richard Henderson
2001-11-16 3:13 ` Richard Henderson
2001-11-16 3:42 ` Corey Minyard
2001-11-16 8:54 ` Bryce McKinlay
2001-11-26 0:47 ` Bryce McKinlay
2001-11-25 20:27 ` Corey Minyard
2001-11-17 1:33 ` Corey Minyard
2001-11-17 6:09 ` Corey Minyard
2001-11-26 10:22 ` Corey Minyard
2001-11-17 11:51 ` Richard Henderson
2001-11-17 15:20 ` Corey Minyard
2001-11-26 14:45 ` Corey Minyard
2001-11-18 5:15 ` Corey Minyard
2001-11-18 8:59 ` Richard Henderson
2001-11-19 2:58 ` Corey Minyard
2001-11-19 3:11 ` Richard Henderson
2001-11-19 5:36 ` Corey Minyard
2001-11-19 7:48 ` Richard Henderson
2001-11-19 7:58 ` Corey Minyard [this message]
2001-11-19 9:43 ` Richard Henderson
2001-11-19 12:44 ` Corey Minyard
2001-11-19 13:53 ` Richard Henderson
2001-11-20 11:20 ` Corey Minyard
2001-11-27 19:57 ` Corey Minyard
2001-11-27 15:31 ` Richard Henderson
2001-11-27 15:04 ` Corey Minyard
2001-11-27 14:27 ` Richard Henderson
2001-11-27 13:25 ` Corey Minyard
2001-11-27 12:58 ` Richard Henderson
2001-11-27 11:51 ` Corey Minyard
2001-11-27 10:29 ` Richard Henderson
2001-11-27 10:08 ` Corey Minyard
2001-11-26 23:18 ` Richard Henderson
2001-11-26 17:15 ` Corey Minyard
2001-11-26 13:49 ` Richard Henderson
2001-11-26 8:58 ` Corey Minyard
2001-11-25 17:58 ` Richard Henderson
2001-11-25 15:06 ` Corey Minyard
2001-11-15 18:47 Richard Kenner
2001-11-25 15:13 ` Richard Kenner
2001-11-15 19:05 dewar
2001-11-25 15:20 ` dewar
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=3C040513.60803@acm.org \
--to=minyard@acm.org \
--cc=gcc@gcc.gnu.org \
--cc=rth@redhat.com \
/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).