[libcxx-commits] [PATCH] D65041: Change requirements on linear_congruential_engine

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a reviewer: libc++.
ldionne added inline comments.


================
Comment at: libcxx/include/random:1677
+{
+    static_assert(__a != 0 || __m != 0 ||
+                  !_MightOverflow ||
----------------
zoecarver wrote:
> This `static_assert` is redundant. Maybe I should remove it. 
Can this be simplified to `static_assert(!_MightOverflow || _OverflowOK || _ShrageOk, "");`?


Repository:
  rCXX libc++

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

https://reviews.llvm.org/D65041