[PATCH] D33080: [Libc++] Use #pragma push_macro/pop_macro to better handle min/max on Windows
Saleem Abdulrasool via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun May 14 12:38:05 PDT 2017
compnerd requested changes to this revision.
compnerd added a comment.
This revision now requires changes to proceed.
I think that we should sink the `min`/`max` checks into `__undef_macros`. I don't like the idea of littering that check everywhere.
================
Comment at: include/__config:1218
+# if defined(_LIBCPP_COMPILER_MSVC)
+# define _LIBCPP_PUSH_MACROS \
+ __pragma(push_macro("min")) \
----------------
clang-format these please? It does a nice job of aligning the `\`.
https://reviews.llvm.org/D33080
More information about the cfe-commits
mailing list