[PATCH] D38808: [CMake] Allow LLVM_ENABLE_LTO to use MSVC LTCG

[Bob Haarman via Phabricator via llvm-commits]

inglorion requested changes to this revision.
inglorion added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/cmake/modules/HandleLLVMOptions.cmake:781
+      # If we have clang-cl + lld-link, we can use Thin or Full
+      set(${style} ${LLVM_ENABLE_LTO} PARENT_SCOPE)
+      return()
----------------
Can you set ${style} to "OFF", "FULL", "MSVC", and "THIN" only? That way, dispatching on it won't have to worry about case and "ON" being an alias for "FULL". Also in the other passthrough case further down.


https://reviews.llvm.org/D38808