[libcxx-commits] [PATCH] D68840: [libc++][P0980] Marked member functions move/copy/assign of char_traits constexpr.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne marked an inline comment as done.
ldionne added inline comments.


================
Comment at: libcxx/test/std/strings/char.traits/char.traits.specializations/char.traits.specializations.char/assign3.pass.cpp:20
 
-int main(int, char**)
+TEST_CONSTEXPR_CXX20 bool test()
 {
----------------
ldionne wrote:
> Did you forget to include the change to `test_macros.h` (or whatever file) where you add `TEST_CONSTEXPR_CXX20`? I can't find it.
> 
> Also, I would suggest `TEST_CONSTEXPR_AFTER_CXX17`, to mirror the checks we usually make (which are always of the form `#if STD > 17`.
> Did you forget to include the change to `test_macros.h` (or whatever file) where you add `TEST_CONSTEXPR_CXX20`? I can't find it.

Nevermind this, I just saw the other review. My bad.


> Also, I would suggest `TEST_CONSTEXPR_AFTER_CXX17`, to mirror the checks we usually make (which are always of the form `#if STD > 17`.

Forget this, I'll comment on D68837 instead.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68840