On Fri, Jul 6, 2018 at 11:45 AM Kugan Vivekanandarajah
<kugan.vivekanandara...@linaro.org> wrote:
>
> Hi Richard,
>
> > It was rewrite_to_non_trapping_overflow available  in tree.h.  Thus
> > final value replacement
> > could use that before gimplifying instead of using 
> > rewrite_to_defined_overflow
> Thanks.
>
> Is the attached patch OK? I am testing this on x86_64-linux-gnu and if
> there is no new regressions.

Please clean up the control flow to

  if (...)
    def = rewrite_to_non_trapping_overflow (def);
  def = force_gimple_operand_gsi (&gsi, def, false, NULL_TREE,
                                        true, GSI_SAME_STMT);

OK with that change.
Richard.

> Thanks,
> Kugan
>
> gcc/ChangeLog:
>
> 2018-07-06  Kugan Vivekanandarajah  <kug...@linaro.org>
>
>     * tree-scalar-evolution.c (final_value_replacement_loop): Use
>     rewrite_to_non_trapping_overflow instead of rewrite_to_defined_overflow.

Reply via email to