[libcxx-commits] [PATCH] D99516: [libc++] Avoid <climits> dependency in <thread>

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added inline comments.


================
Comment at: libcxx/include/thread:381
     }
 }
 
----------------
Seems reasonable to me, but I do wonder:
(1) Why does the old code even need to drag in `long double`? What's wrong with regular `double`? (This is in generic code where 99% of the time `__d` will just be plain old `std::chrono::seconds` or `std::chrono::milliseconds`, which are denominated in `long long`.)
(2) What do the relevant unit tests look like?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99516/new/

https://reviews.llvm.org/D99516