Re: [PATCH] PHIOPT: Fix diamond case of match_simplify_replacement

2023-05-04 Thread Richard Biener via Gcc-patches
On Thu, May 4, 2023 at 11:57 PM Andrew Pinski via Gcc-patches wrote: > > So it turns out I messed checking which edge was true/false for the diamond > form. The edges, e0 and e1 here are edges from the merge block but the > true/false edges are from the conditional block and with diamond/threeway,

[PATCH] PHIOPT: Fix diamond case of match_simplify_replacement

2023-05-04 Thread Andrew Pinski via Gcc-patches
So it turns out I messed checking which edge was true/false for the diamond form. The edges, e0 and e1 here are edges from the merge block but the true/false edges are from the conditional block and with diamond/threeway, there is a bb inbetween on both edges. Most of the time, the check that was i