On 10/3/19 5:58 AM, Vladislav Ivanishin wrote:
> This refactoring makes gsi_next_nonvirtual_phi() always advance the
> gimple phi iterator (regardless of whether the current phi is virtual or
> not). It matches the behavior of other gsi_next functions.
>
> It can be applied independently of the other patches (but patch 3
> depends on this one).
>
> I regstrapped this patch separately on x86_64-pc-linux-gnu. Ok?
>
>
> 0001-Make-gsi_next_nonvirtual_phi-do-what-one-expects.patch
>
> From cbf3fa8408f54baffbec79d304d930e17aa7231c Mon Sep 17 00:00:00 2001
> From: Vladislav Ivanishin <v...@ispras.ru>
> Date: Thu, 29 Aug 2019 14:41:06 +0300
> Subject: [PATCH 1/3] Make gsi_next_nonvirtual_phi do what one expects
>
> gcc/:
>
> * gimple-iterator.h (gsi_next_nonvirtual_phi): Change the semantics to
> match that of other gsi_next_* functions. Adjust the comment.
> (gsi_start_nonvirtual_phis): New function.
> * ipa-icf.c (sem_function::compare_phi_node): Update uses of
> gsi_next_nonvirtual_phi accordingly. (No functional change.)
OK
jeff