Re: [PATCH 3/7] Duplicate the range information of the phi onto the new ssa_name

2021-06-25 Thread Andrew Pinski via Gcc-patches
On Thu, Jun 24, 2021 at 8:17 AM Jeff Law via Gcc-patches wrote: > > > > On 6/23/2021 4:19 PM, apinski--- via Gcc-patches wrote: > > From: Andrew Pinski > > > > Since match_simplify_replacement uses gimple_simplify, there is a new > > ssa name created sometimes and then we go and replace the phi e

Re: [PATCH 3/7] Duplicate the range information of the phi onto the new ssa_name

2021-06-24 Thread Jeff Law via Gcc-patches
On 6/23/2021 4:19 PM, apinski--- via Gcc-patches wrote: From: Andrew Pinski Since match_simplify_replacement uses gimple_simplify, there is a new ssa name created sometimes and then we go and replace the phi edge with this new ssa name, the range information on the phi is lost. Placing this

[PATCH 3/7] Duplicate the range information of the phi onto the new ssa_name

2021-06-23 Thread apinski--- via Gcc-patches
From: Andrew Pinski Since match_simplify_replacement uses gimple_simplify, there is a new ssa name created sometimes and then we go and replace the phi edge with this new ssa name, the range information on the phi is lost. Placing this in replace_phi_edge_with_variable is the best option instead