This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, RTL]: Fix PR 63483, Scheduler performs Invalid move of aliased memory reference
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Henderson <rth at redhat dot com>
- Date: Wed, 8 Oct 2014 13:56:56 +0200
- Subject: Re: [PATCH, RTL]: Fix PR 63483, Scheduler performs Invalid move of aliased memory reference
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4YiCbsbH7LH=Dt=zN-zb76izkhH703y7_GOcQTD8Ccvmw at mail dot gmail dot com> <alpine dot LSU dot 2 dot 11 dot 1410081249390 dot 20733 at zhemvz dot fhfr dot qr>
On Wed, Oct 8, 2014 at 12:51 PM, Richard Biener <rguenther@suse.de> wrote:
>> This message revives an old thread [1], where the miscompilation of gfortran
>> on alpha was found that that resulted in:
[...]
> As said in the audit trail of the bugreport I think that the caller
> of alpha_set_memflags is wrong in applying MEM flags from the _source_
> operand to MEMs generated for the RMW sequence for the destination.
>
> It would be better to fix that instead.
Please see comment #6 of the referred PR [1] for further analysis and
ammended testcase. The testcase and analysis will show a "native" read
passing possibly aliasing store.
[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63483#c6
Uros.