This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR79201 (half-way)
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Richard Biener <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 11 May 2017 14:15:46 +0200
- Subject: Re: [PATCH] Fix PR79201 (half-way)
- Authentication-results: sourceware.org; auth=none
- References: <alpine.LSU.2.20.1704241053350.24645@zhemvz.fhfr.qr> <alpine.LSU.2.20.1704251122390.24645@zhemvz.fhfr.qr>
Hi Richard,
> On Mon, 24 Apr 2017, Richard Biener wrote:
>>
>> One issue in PR79201 is that we don't sink pure/const calls which is
>> what the following simple patch fixes.
>>
>> Bootstrap and regtest running on x86_64-unknown-linux-gnu.
>
> Needed some gimple_assign_lhs -> gimple_get_lhs adjustments and
> adjustment of gcc.target/i386/pr22152.c where we now sink the
> assignment out of the pointless loop. Not sure what the original
> bug was about (well, reg allocation) so I simply disabled sinking
> for it.
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
>
> Richard.
>
> 2017-04-25 Richard Biener <rguenther@suse.de>
>
> PR tree-optimization/79201
> * tree-ssa-sink.c (statement_sink_location): Handle calls.
>
> * gcc.dg/tree-ssa/ssa-sink-16.c: New testcase.
> * gcc.target/i386/pr22152.c: Disable sinking.
however, gcc.target/i386/pr22152.c FAILs now for 32-bit:
FAIL: gcc.target/i386/pr22152.c scan-assembler-times movq[ \\\\t]+[^\\n]*%mm 1
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University