[libcxx-commits] [PATCH] D126119: [libc++] Enable more warnings in the test suite

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/test/std/concepts/concepts.lang/concept.constructible/constructible_from.compile.pass.cpp:109-111
-  test<void (Empty::*)(const int&) throw()>();
-  test<void (Empty::*)(const int&) throw(),
-       void(Empty::*)(const int&) noexcept(true)>();
----------------
philnik wrote:
> @Mordante These test-cases look completely redundant to me. Do you know why you added them?
What do you mean with redundant?


================
Comment at: libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp:26
     std::quick_exit(0);
-    return 0;
 }
----------------
Not your change, but it looks like we're not testing anything in this test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126119