This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Correct fix for scheduler bug PR11320
- From: Bernd Schmidt <bernds at codesourcery dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Steve Ellcey <sje at cup dot hp dot com>, "Vladimir N. Makarov" <vmakarov at redhat dot com>
- Date: Thu, 14 Jul 2011 14:14:41 +0200
- Subject: Re: Correct fix for scheduler bug PR11320
- References: <4E1EBEFE.3080002@codesourcery.com> <201107141357.15147.ebotcazou@adacore.com>
On 07/14/11 13:57, Eric Botcazou wrote:
>> The real problem here is that the ia64 backend lies to the rest of the
>> compiler; it produces a load instruction without showing a MEM anywhere
>> in the instruction pattern. Hence, the following patch, which reverts
>> the bogus scheduler changes and adds a MEM to a pattern in ia64.md.
>
> This is probably the root cause of the problem, indeed. But you don't revert
> everything so, if this isn't an oversight, then the ChangeLog is incorrect.
> And there is another change in sched-deps.c not mentioned in the ChangeLog.
Well, the actual code has completely changed in the meantime. All the
hunks of the original patch failed :) I can write a new ChangeLog entry
if that seems important.
Any particular bits you still see that don't get reverted with this patch?
Bernd