Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Martin Sebor via Gcc-patches
On 8/10/20 2:08 PM, Andrew MacLeod wrote: On 8/10/20 2:46 PM, Martin Sebor wrote: On 8/10/20 11:50 AM, Andrew MacLeod wrote: On 8/10/20 12:35 PM, Martin Sebor via Gcc-patches wrote: On 8/10/20 5:47 AM, Aldy Hernandez wrote: int_range is the type which allows for up to X subranges. calcula

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Andrew MacLeod via Gcc-patches
On 8/10/20 2:46 PM, Martin Sebor wrote: On 8/10/20 11:50 AM, Andrew MacLeod wrote: On 8/10/20 12:35 PM, Martin Sebor via Gcc-patches wrote: On 8/10/20 5:47 AM, Aldy Hernandez wrote: int_range is the type which allows for up to X subranges. calculations will be merged to fit within X subran

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Andrew MacLeod via Gcc-patches
On 8/10/20 2:46 PM, Martin Sebor wrote: On 8/10/20 11:50 AM, Andrew MacLeod wrote: On 8/10/20 12:35 PM, Martin Sebor via Gcc-patches wrote: On 8/10/20 5:47 AM, Aldy Hernandez wrote: On 8/6/20 9:30 PM, Martin Sebor wrote: On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: +  // Remo

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Martin Sebor via Gcc-patches
On 8/10/20 11:50 AM, Andrew MacLeod wrote: On 8/10/20 12:35 PM, Martin Sebor via Gcc-patches wrote: On 8/10/20 5:47 AM, Aldy Hernandez wrote: On 8/6/20 9:30 PM, Martin Sebor wrote: On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: +  // Remove the unknown parts of a multi-range. + 

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Aldy Hernandez via Gcc-patches
Yes, the goal is that anything that may take multi ranges be rewritten to use an irange * and use the API exclusively. Then when multi ranges are passed down eventually, things will work transparently. Aldy On Mon, Aug 10, 2020, 19:50 Andrew MacLeod wrote: > On 8/10/20 12:35 PM, Martin Sebor v

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Andrew MacLeod via Gcc-patches
On 8/10/20 12:35 PM, Martin Sebor via Gcc-patches wrote: On 8/10/20 5:47 AM, Aldy Hernandez wrote: On 8/6/20 9:30 PM, Martin Sebor wrote: On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: +  // Remove the unknown parts of a multi-range. +  // This will transform [5,10][20,MAX] into

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Martin Sebor via Gcc-patches
On 8/10/20 5:47 AM, Aldy Hernandez wrote: On 8/6/20 9:30 PM, Martin Sebor wrote: On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: +  // Remove the unknown parts of a multi-range. +  // This will transform [5,10][20,MAX] into [5,10]. Is this comment correct?  Wouldn't this result i

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-10 Thread Aldy Hernandez via Gcc-patches
On 8/6/20 9:30 PM, Martin Sebor wrote: On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: +  // Remove the unknown parts of a multi-range. +  // This will transform [5,10][20,MAX] into [5,10]. Is this comment correct?  Wouldn't this result in returning smaller sizes than the actual

Re: [PATCH] Rewrite get_size_range for irange API.

2020-08-06 Thread Martin Sebor via Gcc-patches
On 8/6/20 8:53 AM, Aldy Hernandez via Gcc-patches wrote: [Martin, does this sound reasonable to you?] It mostly makes sense to me except one part: The following patch converts get_size_range to the irange API, thus removing the use of VR_ANTI_RANGE. This was a bit tricky because of the gymn