[libcxx-commits] [PATCH] D151205: [libc++][spaceship] Additional tests for `operator<=>` `map` and `multimap`

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added a comment.

In D151205#4364787 <https://reviews.llvm.org/D151205#4364787>, @H-G-Hristov wrote:

> Try to fix CI: Ignore `expected-error` message with `{{}}` syntax due to differing error message locally vs remotely (on CI).

You probably want to update your local compiler then. We only support Clang 15, 16 and trunk.



================
Comment at: libcxx/test/std/containers/associative/map/map.nonmember/compare.three_way.verify.cpp:21
+
+#include "test_allocator.h"
+
----------------
This include doesn't seem to be used,


================
Comment at: libcxx/test/std/containers/associative/map/map.nonmember/compare.three_way.verify.cpp:28
+    std::map<int, int, std::greater<int>> s2;
+    // expected-error@*:* 2 {{invalid operands to binary expression}}
+    s1 <=> s2;
----------------
IMO it would be better to have two separate `// expected-error`s to make sure they are actually generated by the two lines below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151205