[PATCH] D22017: [libcxx] [test] Work around MSVC's non-Standard ABI for enums.

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 24 17:04:01 PDT 2016


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

> in underlying_type.pass.cpp, I'm simply marking the affected static_asserts as libcxx-specific (this probably won't be sufficient when you try to get the Clang/LLVM/libcxx stack running on Windows). In make_signed.pass.cpp and make_unsigned.pass.cpp, I'm giving the enum an explicitly specified underlying type, so the tests below can remain unaffected.


I think it would be better to check if we are on Windows, and test for the respected result accordingly.


https://reviews.llvm.org/D22017





More information about the cfe-commits mailing list