* Jakub Jelinek: > On Wed, Nov 03, 2021 at 05:27:42PM +0100, Florian Weimer wrote: >> + /* S3. See __condvar_load_64_relaxed. */ > > Shouldn't that be See __atomic_wide_counter_load_relaxed ?
I'm going to fix the stale reference, thanks. >> + if (((l >> 31) > 0) && ((h >> 31) > 0)) > > Any reason not to write it as (int) l < 0 && (int) h < 0? This requires a GCC extension (which active by default, even without -fwrapv). Obviously we relay on this extension in many other place, but still … Thanks, Florian