[PATCH] D37376: [libcxx] Fix libc++experimental build on Windows
Saleem Abdulrasool via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 11 09:07:21 PDT 2017
compnerd requested changes to this revision.
compnerd added inline comments.
This revision now requires changes to proceed.
================
Comment at: lib/CMakeLists.txt:189
+macro(add_msvcrt_defs_if_needed target)
+ if(WIN32 AND NOT MINGW)
----------------
Please use `function` rather than `macro`. There is no need to pollute the namespace here.
https://reviews.llvm.org/D37376
More information about the cfe-commits
mailing list