Skip to content

Commit

Permalink
[libc++] Avoid implicit conversion warning in a <random> test
Browse files Browse the repository at this point in the history
By stashing the computation of `E::max() - E::min()` in a variable, we
avoid the warning introduced in r367497. Note that we use `auto` to
avoid having to deduce the type of the computation, which is not a
problem since Clang provides `auto` as an extension even in C++03 (and
we disable warnings related to using C++11 extensions in the test suite).

llvm-svn: 369429
  • Loading branch information
ldionne committed Aug 20, 2019
1 parent bc2f425 commit c310e5a
Showing 1 changed file with 16 additions and 23 deletions.
Expand Up @@ -19,85 +19,78 @@

int main(int, char**)
{
typedef std::minstd_rand0 E;
auto range = E::max() - E::min();

{
typedef std::minstd_rand0 E;
typedef float F;
E r;
F f = std::generate_canonical<F, 0>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef float F;
E r;
F f = std::generate_canonical<F, 1>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef float F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits - 1>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef float F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef float F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits + 1>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}

{
typedef std::minstd_rand0 E;
typedef double F;
E r;
F f = std::generate_canonical<F, 0>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef double F;
E r;
F f = std::generate_canonical<F, 1>(r);
assert(f == truncate_fp((16807 - E::min()) / (E::max() - E::min() + F(1))));
assert(f == truncate_fp((16807 - E::min()) / (range + F(1))));
}
{
typedef std::minstd_rand0 E;
typedef double F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits - 1>(r);
assert(f == truncate_fp(
(16807 - E::min() +
(282475249 - E::min()) * (E::max() - E::min() + F(1))) /
((E::max() - E::min() + F(1)) * (E::max() - E::min() + F(1)))));
(282475249 - E::min()) * (range + F(1))) /
((range + F(1)) * (range + F(1)))));
}
{
typedef std::minstd_rand0 E;
typedef double F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits>(r);
assert(f == truncate_fp(
(16807 - E::min() +
(282475249 - E::min()) * (E::max() - E::min() + F(1))) /
((E::max() - E::min() + F(1)) * (E::max() - E::min() + F(1)))));
(282475249 - E::min()) * (range + F(1))) /
((range + F(1)) * (range + F(1)))));
}
{
typedef std::minstd_rand0 E;
typedef double F;
E r;
F f = std::generate_canonical<F, std::numeric_limits<F>::digits + 1>(r);
assert(f == truncate_fp(
(16807 - E::min() +
(282475249 - E::min()) * (E::max() - E::min() + F(1))) /
((E::max() - E::min() + F(1)) * (E::max() - E::min() + F(1)))));
(282475249 - E::min()) * (range + F(1))) /
((range + F(1)) * (range + F(1)))));
}

return 0;
Expand Down

0 comments on commit c310e5a

Please sign in to comment.