Re: [PR77719] Fix ICE in pp_string, at pretty-print.c:955

[Jeff Law <law@redhat.com>]

On 09/25/2016 08:39 PM, kugan wrote:
>
>
> On 25/09/16 04:50, kugan wrote:
>> Hi,
>>
>> In make_new_ssa_for_def (tree-reassoc.c) we should use gimple_get_lhs to
>> get lhs instead of gimple_assign_lhs as stmt can be builtins too.
>> Attached patch fixes this.
>>
>> Testcase from PR (attached) seems to fail. I dont any fortran so I didnt
>> try fixing it. Any help here is appreciate. The log is:
>
> Here is a patch with the testcase based on the feedback from Joost
> VandeVondele and Dominique d'Humieres. Bootstrapped and regression
> tested on x86_64-linux-gnu with no new regressions.
>
> Is this OK for trunk?
>
> Thanks,
> kugan
>
> gcc/testsuite/ChangeLog:
>
> 2016-09-25  Kugan Vivekanandarajah  <kuganv@linaro.org>
>
>     * gfortran.dg/pr77719.f90: New test.
>
> gcc/ChangeLog:
>
> 2016-09-25  Kugan Vivekanandarajah  <kuganv@linaro.org>
>
>     * tree-ssa-reassoc.c (make_new_ssa_for_def): Use gimple_get_lhs to
> get lhs
>     instead of gimple_assign_lhs as stmt can be builtins too.
>
OK.
jeff