Re: [PATCH] Adjust gimple-ssa-sprintf.c for irange API.

2020-08-04 Thread Martin Sebor via Gcc-patches
On 8/4/20 8:11 AM, Aldy Hernandez wrote: On Tue, Aug 4, 2020 at 3:59 PM Martin Sebor wrote: On 8/4/20 5:21 AM, Aldy Hernandez via Gcc-patches wrote: This is a rather obvious patch, but I'd like a nod before committing. Martin, I've removed your anti-range check, as it is subsumed by the lowe

Re: [PATCH] Adjust gimple-ssa-sprintf.c for irange API.

2020-08-04 Thread Aldy Hernandez via Gcc-patches
On Tue, Aug 4, 2020 at 3:59 PM Martin Sebor wrote: > > On 8/4/20 5:21 AM, Aldy Hernandez via Gcc-patches wrote: > > This is a rather obvious patch, but I'd like a nod before committing. > > > > Martin, I've removed your anti-range check, as it is subsumed by the > > lower_bound/upper_bound code.

Re: [PATCH] Adjust gimple-ssa-sprintf.c for irange API.

2020-08-04 Thread Martin Sebor via Gcc-patches
On 8/4/20 5:21 AM, Aldy Hernandez via Gcc-patches wrote: This is a rather obvious patch, but I'd like a nod before committing. Martin, I've removed your anti-range check, as it is subsumed by the lower_bound/upper_bound code. However, you will have to adapt the code for multi-ranges if desired.

[PATCH] Adjust gimple-ssa-sprintf.c for irange API.

2020-08-04 Thread Aldy Hernandez via Gcc-patches
This is a rather obvious patch, but I'd like a nod before committing. Martin, I've removed your anti-range check, as it is subsumed by the lower_bound/upper_bound code. However, you will have to adapt the code for multi-ranges if desired. For example, you may want to loop through the sub-ranges