Re: [PATCH] forwprop: Further fixes for simplify_rotate [PR108440]

2023-01-19 Thread Aldy Hernandez via Gcc-patches
On 1/19/23 09:41, Jakub Jelinek wrote: + range_query *q = get_range_query (cfun); + if (q == get_global_range_query ()) + q = enable_ranger (cfun); Oh, neat. Clever. I hadn't thought about that. + if (!q->range_of_expr (r, rotcnt, check_range_stmt)) +

Re: [PATCH] forwprop: Further fixes for simplify_rotate [PR108440]

2023-01-19 Thread Richard Biener via Gcc-patches
On Thu, 19 Jan 2023, Jakub Jelinek wrote: > Hi! > > As mentioned in the simplify_rotate comment, for e.g. >((T) ((T2) X << (Y & (B - 1 | ((T) ((T2) X >> ((-Y) & (B - 1 > we already emit >X r<< (Y & (B - 1)) > as replacement. This PR is about the >((T) ((T2) X << Y)) OP ((T) (

[PATCH] forwprop: Further fixes for simplify_rotate [PR108440]

2023-01-19 Thread Jakub Jelinek via Gcc-patches
Hi! As mentioned in the simplify_rotate comment, for e.g. ((T) ((T2) X << (Y & (B - 1 | ((T) ((T2) X >> ((-Y) & (B - 1 we already emit X r<< (Y & (B - 1)) as replacement. This PR is about the ((T) ((T2) X << Y)) OP ((T) ((T2) X >> (B - Y))) ((T) ((T2) X << (int) Y)) OP ((T) ((