Re: [PATCH v2] [libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-09-02 Thread Jonathan Wakely
On Thu, 1 Aug 2024 at 11:44, Alexandre Oliva wrote: > > When we get to test_pr91486_wait_until(), we're about 10s past the > float_steady_clock epoch. This is enough for the 1s delta for the > timeout to come out slightly lower when the futex-less wait_until > converts the deadline from float_ste

Re: [PATCH v2] [libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-08-15 Thread Alexandre Oliva
On Aug 7, 2024, Alexandre Oliva wrote: > On Aug 1, 2024, Alexandre Oliva wrote: >> Each iteration calls float_steady_clock::now() [...] an extra iteration >> will reach 5 and cause the test to fail. >> (Do we really want to use floats, that even with this tweak have >> borderline precision fo

Re: [PATCH v2] [libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-08-07 Thread Alexandre Oliva
On Aug 1, 2024, Alexandre Oliva wrote: > Each iteration calls float_steady_clock::now() [...] an extra iteration > will reach 5 and cause the test to fail. > (Do we really want to use floats, that even with this tweak have > borderline precision for sub-µs vs 1s deltas? Do we want to make sure

[PATCH v2] [libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-08-01 Thread Alexandre Oliva
When we get to test_pr91486_wait_until(), we're about 10s past the float_steady_clock epoch. This is enough for the 1s delta for the timeout to come out slightly lower when the futex-less wait_until converts the deadline from float_steady_clock to __clock_t. So we may wake up a little too early,

Re: [libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-07-29 Thread Alexandre Oliva
On Jul 29, 2024, Alexandre Oliva wrote: > - auto status = f1.wait_for(wait_time); > + auto status __attribute__ (__unused__) = f1.wait_for(wait_time); Sorry, it looks like I posted the patch before refreshing it. Make it: + auto status __attribute__ ((__unused__)) = f1.wait_for(wait_time)

[libstdc++] [testsuite] avoid async.cc loss of precision [PR91486]

2024-07-29 Thread Alexandre Oliva
When we get to test_pr91486_wait_until(), we're about 10s past the float_steady_clock epoch. This is enough for the 1s delta for the timeout to come out slightly lower when the futex-less wait_until converts the deadline from float_steady_clock to __clock_t. So we may wake up a little too early