[PATCH] D54966: Implement P1007R3 `std::assume_aligned`

Roman Lebedev via Phabricator reviews at reviews.llvm.org
Tue Nov 27 12:39:18 PST 2018


lebedev.ri added a comment.

Looks good in general, like i suspected "deceivingly simple" :)
Related: D54589 <https://reviews.llvm.org/D54589>



================
Comment at: include/memory:5591
+_LIBCPP_NODISCARD_ATTRIBUTE 
+constexpr _Tp* assume_aligned(_Tp * __p) noexcept
+{
----------------
Drop `noexcept`?
`— R2, 2018-10-08: Revised wording and removed noexcept following LWG guidance.`


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

https://reviews.llvm.org/D54966





More information about the libcxx-commits mailing list