[ was: Re: [PR65637][PATCH][2/3] Fix inner loop phi in 
expand_omp_for_static_chunk ]

On 15/04/15 15:17, Tom de Vries wrote:
> On 15-04-15 15:10, Tom de Vries wrote:
>> Hi,
>>
>> This patch series fixes PR65637.
>>

<SNIP>

>
> This patch fixes an libgomp.c/autopar-1.c execution failure.
>
> For autopar-1.c, the original loop has a loop phi:
> ...
> # s.5_20 = PHI <s.5_12(4), 0.0(20)>
> ...
>
> After expand_omp_for_static_chunk, there's an inner and an outer loop.
> The outer
> loop phi is:
> ...
> # s.5_11 = PHI <0.0(15), s.5_12(21)>
> ...
>
> and the inner loop phi is:
> ...
> # s.5_20 = PHI <s.5_12(4), 0.0(20)>
> ...
>
> The inner loop phi should not have 0.0 as argument, but the result of
> the outer
> loop phi, like this:
> ...
> # s.5_20 = PHI <s.5_12(4), s.5_11(20)>
> ...
>
> This patch fixes the inner loop phi, and allows the autopar-1.c
> execution test
> to pass.
>

This updated patch includes a test-case. It also handles simple latches.

OK for trunk?

Thanks,
- Tom