[PATCH] D26903: [libcxx] Add <variant> tests (but not implementation)

Stephan T. Lavavej via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 22 16:41:41 PST 2016


STL_MSFT added inline comments.


================
Comment at: test/std/utilities/variant/variant.helpers/variant_alternative.pass.cpp:33
+void test() {
+    static_assert(std::is_same_v<
+        typename std::variant_alternative<I, V>::type, E>, "");
----------------
EricWF wrote:
> STL_MSFT wrote:
> > You aren't directly including <type_traits> (not sure if "variant_test_helpers.hpp" is).
> Fixing this. Does your `<variant>` somehow not drag in `<type_traits>`?
Of course <variant> drags in <type_traits>, I'm just being nitpicky :-)


https://reviews.llvm.org/D26903





More information about the cfe-commits mailing list