Re: [PATCH 5/5] Sanitize legacy_{lower,upper}_bound

2023-03-01 Thread Aldy Hernandez via Gcc-patches
Similar to my num_pairs comment. If we're fixing missed optimizations, then yes. Otherwise I'd prefer to remove it soon. That being said, you are the release manager and if you feel strongly about this I would defer to you. Aldy On 2/28/23 14:47, Richard Biener via Gcc-patches wrote:

[PATCH 5/5] Sanitize legacy_{lower,upper}_bound

2023-02-28 Thread Richard Biener via Gcc-patches
* value-range.h (irange::legacy_lower_bound): Remove pair argument. (irange::legacy_upper_bound): Likewise. (irange::lower_bound): Commonize asserts, adjust. (irange::upper_bound): Likewise. * value-range.cc (irange::legacy_lower_bound): Remove