Re: [PATCH 2/7] Do not query SCEV in range_of_phi unless dominators are available.

2021-09-23 Thread Aldy Hernandez via Gcc-patches
On 9/22/21 10:05 AM, Richard Biener wrote: On Tue, Sep 21, 2021 at 7:17 PM Aldy Hernandez via Gcc-patches wrote: On 9/21/21 7:05 PM, Andrew MacLeod wrote: On 9/21/21 12:53 PM, Aldy Hernandez wrote: SCEV won't work without dominators and we can get called without dominators from

Re: [PATCH 2/7] Do not query SCEV in range_of_phi unless dominators are available.

2021-09-22 Thread Richard Biener via Gcc-patches
On Tue, Sep 21, 2021 at 7:17 PM Aldy Hernandez via Gcc-patches wrote: > > > > On 9/21/21 7:05 PM, Andrew MacLeod wrote: > > On 9/21/21 12:53 PM, Aldy Hernandez wrote: > >> SCEV won't work without dominators and we can get called without > >> dominators from debug_ranger. > >> > >> Another option

Re: [PATCH 2/7] Do not query SCEV in range_of_phi unless dominators are available.

2021-09-21 Thread Aldy Hernandez via Gcc-patches
On 9/21/21 7:05 PM, Andrew MacLeod wrote: On 9/21/21 12:53 PM, Aldy Hernandez wrote: SCEV won't work without dominators and we can get called without dominators from debug_ranger. Another option would be to rename scev_initialized_p to something like scev_available_p and move the check

Re: [PATCH 2/7] Do not query SCEV in range_of_phi unless dominators are available.

2021-09-21 Thread Andrew MacLeod via Gcc-patches
On 9/21/21 12:53 PM, Aldy Hernandez wrote: SCEV won't work without dominators and we can get called without dominators from debug_ranger. Another option would be to rename scev_initialized_p to something like scev_available_p and move the check there. For now, this will do. Committed.

[PATCH 2/7] Do not query SCEV in range_of_phi unless dominators are available.

2021-09-21 Thread Aldy Hernandez via Gcc-patches
SCEV won't work without dominators and we can get called without dominators from debug_ranger. Another option would be to rename scev_initialized_p to something like scev_available_p and move the check there. For now, this will do. Committed. gcc/ChangeLog: * gimple-range-fold.cc