[libcxx-commits] [PATCH] D132667: [libc++] Inlines format_error for clang-cl DLL.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/include/__format/format_error.h:35
+  // TODO FMT Remove when format is no longer experimental.
+  // Avoids linker errors when building the Clang Windows DLL which doesn't support the experimental library
+#  ifndef _LIBCPP_INLINE_FORMAT_ERROR_DTOR
----------------
mstorsjo wrote:
> Nitpick, "Clang-cl", or "MSVC mode" or something like that, but not plain "Clang".
Thanks for the comment! I'll fix it when landing the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132667