]> granicus.if.org Git - llvm/commitdiff
[cmake] Note future cleanup in comment. NFC
authorShoaib Meenai <smeenai@fb.com>
Thu, 31 Jan 2019 20:32:45 +0000 (20:32 +0000)
committerShoaib Meenai <smeenai@fb.com>
Thu, 31 Jan 2019 20:32:45 +0000 (20:32 +0000)
CMake 3.6 introduced CMAKE_TRY_COMPILE_PLATFORM_VARIABLES, which solves
precisely the problem that necessitated init_user_prop, so we can switch
over whenever we bump our minimum CMake requirement.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@352790 91177308-0d34-0410-b5e6-96231b3b80d8

cmake/platforms/WinMsvc.cmake

index f625d0e3c052412734b4f56af6722bada1ef4df3..d038df9795131fb248e63859e48735d6159524bc 100644 (file)
@@ -92,7 +92,9 @@
 # won't see the value of any arguments the user passed via -D.  Since these are
 # necessary to properly configure MSVC in both the top-level configuration as well as
 # all feature-test invocations, we set environment variables with the values so that
-# these environments get inherited by child invocations.
+# these environments get inherited by child invocations. We can switch to
+# CMAKE_TRY_COMPILE_PLATFORM_VARIABLES once our minimum supported CMake version
+# is 3.6 or greater.
 function(init_user_prop prop)
   if(${prop})
     set(ENV{_${prop}} "${${prop}}")