[libcxx-commits] [PATCH] D60666: Fix some constexpr members in array<T, 0>

[Zoe Carver via Phabricator via libcxx-commits]

zoecarver created this revision.
zoecarver added reviewers: rsmith, EricWF, ldionne.
Herald added subscribers: libcxx-commits, dexonsmith.

This fixes bug 40124 <https://bugs.llvm.org/show_bug.cgi?id=40124> based on Richard Smith's suggestion. As he pointed out in the bug comments, this implementation does not take up any more space.

Currently, `__wrapper` is trivially destructible (I think there is an issue about this for C++20, but I cannot find any specifications about it in the standard. Feel free to post a link, and I can update the patch).

This patch relies on issue 2157 <https://cplusplus.github.io/LWG/issue2157> which ensures that `_Tp` is `DefaultConstructible` (maybe I should add a check for this?).


Repository:
  rCXX libc++

https://reviews.llvm.org/D60666

Files:
  include/array
  test/std/containers/sequences/array/size_and_alignment.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60666.195064.patch
Type: text/x-patch
Size: 6357 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190414/d73a784e/attachment.bin>