Re: [PATCH] PR tree-optimization/107570 - Reset SCEV after folding in VRP.

2023-02-03 Thread Richard Biener via Gcc-patches
> Am 03.02.2023 um 16:54 schrieb Andrew MacLeod : > >  >> On 2/2/23 07:22, Richard Biener wrote: >>> On Wed, Feb 1, 2023 at 7:12 PM Andrew MacLeod via Gcc-patches >>> wrote: >>> We can reset SCEV after we fold, then SCEVs cache shouldn't have >>> anything in it when we go to remove ssa-names

Re: [PATCH] PR tree-optimization/107570 - Reset SCEV after folding in VRP.

2023-02-03 Thread Andrew MacLeod via Gcc-patches
On 2/2/23 07:22, Richard Biener wrote: On Wed, Feb 1, 2023 at 7:12 PM Andrew MacLeod via Gcc-patches wrote: We can reset SCEV after we fold, then SCEVs cache shouldn't have anything in it when we go to remove ssa-names in remove_unreachable(). We were resetting it later sometimes if we were p

Re: [PATCH] PR tree-optimization/107570 - Reset SCEV after folding in VRP.

2023-02-02 Thread Richard Biener via Gcc-patches
On Wed, Feb 1, 2023 at 7:12 PM Andrew MacLeod via Gcc-patches wrote: > > We can reset SCEV after we fold, then SCEVs cache shouldn't have > anything in it when we go to remove ssa-names in remove_unreachable(). > > We were resetting it later sometimes if we were processing the array > bounds warni

[PATCH] PR tree-optimization/107570 - Reset SCEV after folding in VRP.

2023-02-01 Thread Andrew MacLeod via Gcc-patches
We can reset SCEV after we fold, then SCEVs cache shouldn't have anything in it when we go to remove ssa-names in remove_unreachable(). We were resetting it later sometimes if we were processing the array bounds warning, so I removed that call and just always reset it now. Bootstraps on x86_6