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

Louis Dionne via Phabricator reviews at reviews.llvm.org
Wed Nov 28 07:26:33 PST 2018


ldionne marked an inline comment as done.
ldionne added inline comments.


================
Comment at: test/std/utilities/memory/ptr.align/assume_aligned.pass.cpp:28
+{
+	ASSERT_SAME_TYPE(T*, decltype(std::assume_aligned<1, T>(p)));
+	assert((p == std::assume_aligned<1>(p)));
----------------
mclow.lists wrote:
> ldionne wrote:
> > mclow.lists wrote:
> > > Needs a `LIBCPP_ASSERT_NOEXCEPT(std::assume_aligned<1, T>(p))`
> > This check needs to go into `test/libcxx/<...>`.
> I don't think we need an entire test just for one line.
> We have that macro specifically for libc++-specific assertions.
> 
SGTM


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

https://reviews.llvm.org/D54966





More information about the libcxx-commits mailing list