Re: get_range_query vs NULL argument

2023-02-15 Thread Richard Biener via Gcc
On Wed, Feb 15, 2023 at 11:31 PM Andrew MacLeod via Gcc wrote: > > > On 2/15/23 14:50, Andrew Pinski wrote: > > Hi, > >While fixing PR 108354, I came across that > > ssa_name_has_boolean_range calls get_range_query with cfun as the > > argument but sometimes while in IPA passes cfun is current

Re: get_range_query vs NULL argument

2023-02-15 Thread Andrew MacLeod via Gcc
On 2/15/23 14:50, Andrew Pinski wrote: Hi, While fixing PR 108354, I came across that ssa_name_has_boolean_range calls get_range_query with cfun as the argument but sometimes while in IPA passes cfun is currently nullptr. The question should we check the argument before calling get_range_que

get_range_query vs NULL argument

2023-02-15 Thread Andrew Pinski via Gcc
Hi, While fixing PR 108354, I came across that ssa_name_has_boolean_range calls get_range_query with cfun as the argument but sometimes while in IPA passes cfun is currently nullptr. The question should we check the argument before calling get_range_query or is it a valid thing to call it with a

Re: POWER __builtin_add_overflow/__builtin_mul_overflow with u64

2023-02-15 Thread Segher Boessenkool
Hi! On Tue, Feb 14, 2023 at 09:23:55AM +0100, Jakub Jelinek wrote: > CCing Segher and David on this. > rs6000 indeed doesn't implement {,u}{add,sub,mul}v4_optab for > any mode and thus leaves it to the generic code. Yes. Can we do better than the generic code, for those? > On Tue, Feb 14, 2023