[libcxx-commits] [PATCH] D126823: [libc++] Support int8_t and uint8_t in integer distributions as an extension

[Louis Dionne via Phabricator via libcxx-commits]

ldionne updated this revision to Diff 446497.
ldionne added a comment.
Herald added a subscriber: mgorny.

Rebase, try to fix CI (this was much more involved than I thought).

In particular, I left a bunch of TODOs around the quality of negative_binomial (which is really on poisson_distribution,
since that's what negative_binomial uses). I am out of time for figuring out how to further improve poisson_distribution
for small types. I tried various things and nothing really worked for `int8_t` -- I think a rewrite of the algorithm might
be necessary, but that would be an ABI break and a larger project.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126823

Files:
  libcxx/docs/ReleaseNotes.rst
  libcxx/docs/UsingLibcxx.rst
  libcxx/include/CMakeLists.txt
  libcxx/include/__random/binomial_distribution.h
  libcxx/include/__random/clamp_to_integral.h
  libcxx/include/__random/discrete_distribution.h
  libcxx/include/__random/geometric_distribution.h
  libcxx/include/__random/is_valid.h
  libcxx/include/__random/negative_binomial_distribution.h
  libcxx/include/__random/poisson_distribution.h
  libcxx/include/__random/uniform_int_distribution.h
  libcxx/include/module.modulemap.in
  libcxx/include/random
  libcxx/test/libcxx/numerics/clamp_to_integral.pass.cpp
  libcxx/test/libcxx/numerics/rand/rand.req.urng/valid_int_type.verify.cpp
  libcxx/test/libcxx/private_headers.verify.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.bern/rand.dist.bern.bin/eval.pass.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.bern/rand.dist.bern.geo/eval.pass.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.bern/rand.dist.bern.negbin/eval.pass.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.pois/rand.dist.pois.poisson/eval.pass.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.samp/rand.dist.samp.discrete/eval.pass.cpp
  libcxx/test/std/numerics/rand/rand.dist/rand.dist.uni/rand.dist.uni.int/eval.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D126823.446497.patch
Type: text/x-patch
Size: 65968 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220721/40d69226/attachment-0001.bin>