[libcxx-commits] [PATCH] D116344: [libc++] [ABI BREAK] Conform lognormal_distribution::param_type.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

SGTM, but I'm not too familiar with this part of the library.



================
Comment at: libcxx/include/__random/lognormal_distribution.h:37
+        result_type __m_;
+        result_type __s_;
     public:
----------------
If I'm correct we go from `3 * sizeof(result_type) + sizeof(bool)` to `2 * sizeof(result_type)`. So it would be possible use an ABI macro. Right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116344