On Tue, 5 Nov 2019, Andre Vieira (lists) wrote:
> Hi,
>
> When investigating PR92317 I noticed that when we create the skip epilogue
> condition, see ('if (skip_epilog)' in 'vect_do_peeling'), we only copy
> phi-nodes that are not constants in 'slpeel_update_phi_nodes_for_guard2'.
> This can cause problems later when we create the scalar epilogue for this
> epilogue, since if the 'scalar_loop' is not the same as 'loop'
> 'slpeel_tree_duplicate_loop_to_edge_cfg' will expect both to have identical
> single_exit bb's and use that to copy the current_def meta_data of phi-nodes.
>
> This makes sure that is true even if these phi-nodes are constants, fixing
> PR92317. I copied the failing testcase and added the options that originally
> made it fail.
>
> Is this OK for trunk?
OK.
Thanks,
Richard.
