[PATCH] D142983: Add build for Windows on Arm in packaging script

[Pierrick Bouvier via Phabricator via llvm-commits]

pbo-linaro marked 2 inline comments as done.
pbo-linaro added inline comments.


================
Comment at: llvm/utils/release/build_llvm_release.bat:323
+  %cmake_flags% ^
+  -DCMAKE_C_COMPILER=%stage0_bin_dir%/clang-cl.exe ^
+  -DCMAKE_CXX_COMPILER=%stage0_bin_dir%/clang-cl.exe ^
----------------
omjavaid wrote:
> These flags are common to all stage1 builds and can be added to variable that be included to all stage1 builds.
stage0_bin_dir needs to be expanded for every architecture, so it has to be duplicated for all stage1.


================
Comment at: llvm/utils/release/build_llvm_release.bat:332
+  -DCOMPILER_RT_BUILD_SANITIZERS=OFF ^
+  -DLLDB_ENABLE_PYTHON=ON ^
+  -DLLDB_RELOCATABLE_PYTHON=ON ^
----------------
omjavaid wrote:
> These LLDB python related flags should be common for all Stage1 builds too. Should be added to a common variable.
Added them to common_flags for arm64 stage0 and stage1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142983