[libcxx-commits] [PATCH] D80821: [libc++] Fix issues with the triviality of std::array

[Eric Fiselier via Phabricator via libcxx-commits]

EricWF accepted this revision.
EricWF added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libcxx/test/std/containers/sequences/array/array.cons/implicit_copy.pass.cpp:75
+    {
+        typedef std::array<NoDefault, 0> Array;
+        Array array = {};
----------------
These tests aren't changing in behavior, right? You've just re-formated and renamed them?


================
Comment at: libcxx/test/std/containers/sequences/array/array.data/data.pass.cpp:52
         T* p = c.data();
-        LIBCPP_ASSERT(p != nullptr);
+        (void)p;
     }
----------------
I'm OK changing this to `p == nullptr` if that's the behavior we all implement.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80821