[PATCH] D56781: Use response file when generating LLVM-C.dll

[Jakob Bornecrantz via Phabricator via llvm-commits]

Wallbraker added a comment.

This should fix it, doing a build now.



================
Comment at: tools/llvm-shlib/CMakeLists.txt:136
+  # Set this name here and touch up if needed.
+  set(LIBSFILE ${CMAKE_BINARY_DIR}/${CMAKE_CFG_INTDIR}/libllvm-c.args)
+
----------------
stella.stamenova wrote:
> Maybe there's something wrong with my view of this change, but the way I am seeing it, tools/llvm-shlib/CMakeLists.txt appears twice and one of the versions has LIBSFILE and the other one doesn't anymore.
> 
> The way LIBSFILE is defined, it still depends on CMAKE_CFG_INTDIR, so this won't work on Windows with VS.
> 
> Could you check whether you see the change the same way (with two tools/llvm-shlib/CMakeLists.txt) and update it, so that it's clear what the final change is? 
I'm only see one, sorry, not sure what I can do to fix it. My Phabricator-fu is weak.

If I understand CMAKE_CFG_INTDIR correctly it will work, as it's only used in the single config build and in the multi-config build the builder should replace it in the call to python (modulo the one big in this current version.


================
Comment at: tools/llvm-shlib/CMakeLists.txt:138
+
+  if(NOT "${CMAKE_CFG_INTDIR}" STREQUAL ".")
+    foreach(BUILD_MODE ${CMAKE_CONFIGURATION_TYPES})
----------------
stella.stamenova wrote:
> There lots of duplication between the two cases, I think you can simplify this.
Ah I think I see how to do it, I think, I'm not very good at CMake.


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

https://reviews.llvm.org/D56781