[libcxx-commits] [PATCH] D107757: [libcxx] [test] Fix the new_faligned_allocation.pass.cpp test for MSVC

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added inline comments.


================
Comment at: libcxx/test/libcxx/language.support/support.dynamic/new_faligned_allocation.pass.cpp:80
     // And that it mangles the same in C++03 through C++17
+#ifdef _MSC_VER
+    // MSVC uses a different C++ ABI with a different name mangling scheme.
----------------
Shouldn't this use `TEST_COMPILER_C1XX`? Tests generally don't mess with implementation macros like `_MSC_VER` directly, but only through the `test_macros.h` helpers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107757