Re: [PATCH][vect] PR92317: fix skip_epilogue creation for epilogues

2019-11-06 Thread Andre Vieira (lists)
Sorry for the double post, ignore please. On 06/11/2019 10:57, 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_

[PATCH][vect] PR92317: fix skip_epilogue creation for epilogues

2019-11-06 Thread Andre Vieira (lists)
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

Re: [PATCH][vect] PR92317: fix skip_epilogue creation for epilogues

2019-11-06 Thread Richard Biener
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 caus

[PATCH][vect] PR92317: fix skip_epilogue creation for epilogues

2019-11-05 Thread Andre Vieira (lists)
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