Re: [PATCH]middle-end: thread through existing LCSSA variable for alternative exits too [PR113237]

2024-01-09 Thread Richard Biener
On Sun, 7 Jan 2024, Tamar Christina wrote: > Hi All, > > Builing on top of the previous patch, similar to when we have a single exit if > we have a case where all exits are considered early exits and there are > existing > non virtual phi then in order to maintain LCSSA we have to use the

Re: [PATCH]middle-end: thread through existing LCSSA variable for alternative exits too [PR113237]

2024-01-08 Thread Tamar Christina
No, that error is fixed by some earlier patches sent early last week that are awaiting review :) From: Toon Moene Sent: Sunday, January 7, 2024 7:11 PM To: gcc-patches@gcc.gnu.org Subject: Re: [PATCH]middle-end: thread through existing LCSSA variable

Re: [PATCH]middle-end: thread through existing LCSSA variable for alternative exits too [PR113237]

2024-01-07 Thread Toon Moene
On 1/7/24 18:29, Tamar Christina wrote: gcc/ChangeLog: PR tree-optimization/113237 * tree-vect-loop-manip.cc (slpeel_tree_duplicate_loop_to_edge_cfg): Use existing LCSSA variable for exit when all exits are early break. Might that be the same error as I got here when

[PATCH]middle-end: thread through existing LCSSA variable for alternative exits too [PR113237]

2024-01-07 Thread Tamar Christina
Hi All, Builing on top of the previous patch, similar to when we have a single exit if we have a case where all exits are considered early exits and there are existing non virtual phi then in order to maintain LCSSA we have to use the existing PHI variables. We can't simply clear them and just