[libcxx-commits] [PATCH] D123851: [libc++] Change vector<bool>::const_iterator::reference to bool in ABIv2

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/test/std/containers/sequences/vector.bool/iterators.pass.cpp:35
+    ASSERT_SAME_TYPE(ConstIterRefT, std::__bit_const_reference<std::vector<bool> >);
+#endif
     {
----------------
Mordante wrote:
> Interestingly the test `libcxx/test/std/containers/sequences/vector.bool/types.pass.cpp` isn't affected.
> 
> It would be nice when you can add the `const_reference` to the synopsis and add tests for `pointer`, `const_pointer`, and `const_reference`. 
I don't think there is anything portable to test for `pointer` and `const_pointer`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123851