This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] combine: Do another check before splitting a parallel (PR86771)
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: gcc-patches at gcc dot gnu dot org
- Date: Wed, 12 Sep 2018 11:24:09 -0500
- Subject: Re: [PATCH] combine: Do another check before splitting a parallel (PR86771)
- References: <af7b0f7e6b0735b2ad8fbc0de5ad8b1087cc38ff.1534943231.git.segher@kernel.crashing.org>
On Wed, Aug 22, 2018 at 01:19:44PM +0000, Segher Boessenkool wrote:
> When combine splits a resulting parallel into its two SETs, it has to
> place one at i2, and the other stays at i3. This does not work if the
> destination of the SET that will be placed at i2 is modified between
> i2 and i3. This patch fixes it.
>
> Tested on powerpc64-linux {-m32,-m64}, and the reported failing fortran
> testcase on an arm-linux-gnueabi crosscompiler (by inspection).
>
> Committing.
>
>
> 2018-08-22 Segher Boessenkool <segher@kernel.crashing.org>
>
> * combine.c (try_combine): Do not allow splitting a resulting PARALLEL
> of two SETs into those two SETs, one to be placed at i2, if that SETs
> destination is modified between i2 and i3.
I backported this to 8 now.
Segher