[PATCH] D129559: Windows packaging script. Include lld-link, llvm-lib and llvm-windres from stage0

[Hans Wennborg via Phabricator via llvm-commits]

hans added inline comments.


================
Comment at: llvm/utils/release/build_llvm_release.bat:71
 
+if exist %build_dir% rd /s /q %build_dir%
 mkdir %build_dir%
----------------
I don't think we should do this. I think we should exit and let the user decide what to do.


================
Comment at: llvm/utils/release/build_llvm_release.bat:77
 curl -L https://github.com/llvm/llvm-project/archive/%revision%.zip -o src.zip || exit /b
-7z x src.zip || exit /b
+7z x -y src.zip || exit /b
 mv llvm-project-* llvm-project || exit /b
----------------
What are the 7z queries that need "yes"? I don't remember seeing any.


================
Comment at: llvm/utils/release/build_llvm_release.bat:113
+REM Set Python environment
+set PYTHONHOME=%python32_dir%
+set PYTHONPATH=%PYTHONHOME%\lib
----------------
Is there a reason for these new variables, instead of just using `%python32_dir%' "inline" as in the old code?


================
Comment at: llvm/utils/release/build_llvm_release.bat:144
+  -DCMAKE_C_FLAGS="-Wno-nonportable-include-path" ^
+  -DCMAKE_CXX_FLAGS="-Wno-nonportable-include-path" ^
+  -DCMAKE_C_COMPILER=%stage0_bin_dir%/clang-cl.exe ^
----------------
These flags seem new?


================
Comment at: llvm/utils/release/build_llvm_release.bat:149
+  -DCMAKE_AR=%stage0_bin_dir%/llvm-lib ^
+  -DCMAKE_RC=%stage0_bin_dir%/llvm-windres
+set cmake_flags=%all_cmake_flags:\=/%
----------------
Could the LINKER, AR, and RC changes be done separately from this patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129559