[libcxx-commits] [PATCH] D110515: [libc++] [compare] Named comparison functions, is_eq etc.
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Sep 28 09:51:29 PDT 2021
Mordante added inline comments.
================
Comment at: libcxx/test/libcxx/diagnostics/detail.headers/compare/is_eq.module.verify.cpp:16
+// expected-error@*:* {{use of private header from outside its module: '__compare/is_eq.h'}}
+#include <__compare/is_eq.h>
----------------
Quuxplusone wrote:
> @mordante wrote:
> > Would it be possible to handle the comment in test/std/ranges/range.adaptors/range.transform/iterator/compare.pass.cpp:39 in this commit/in a separate commit?
>
> In a separate PR, yes. That was actually how I stumbled onto this. It turns out that that test can't be merely uncommented; it needs some actual "design decision" stuff, so I want to do it in a separate PR. (Namely, `ContiguousView` has iterator type `int*`, so its iterators //are// three-way-comparable; which makes `ThreeWayComparableView` basically redundant. Whereas `contiguous_iterator<int*>` right now is //not// three-way-comparable, but I don't know that we should hard-code too many tests that rely on that being forever true. Anyway, separate PR, etc. :))
When it's indeed more work than uncommenting I also prefer a separate PR.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110515/new/
https://reviews.llvm.org/D110515
More information about the libcxx-commits
mailing list