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

[Muhammad Omair Javaid via Phabricator via llvm-commits]

omjavaid added a comment.

@pbo-linaro I have replied to flang query and also going to issue a build using this script to verify it. Looks good to me. I ll post the result of my build tomorrow.



================
Comment at: llvm/utils/release/build_llvm_release.bat:302
+REM Note that we only enable compiler-rt so we can build the builtins, which are
+REM needed in the next stage when building flang.
+REM We need to build stage0 with clang-cl (for builtins).
----------------
hans wrote:
> We're building flang now?
We have been releasing building flang + mlir for windows on arm since 14.0.0 release. I have tried it on x64 and it passes all ninja check-flang on x64. So I found it reasonable to include flang for all the windows build. 


================
Comment at: llvm/utils/release/build_llvm_release.bat:329
+  -DCPACK_SYSTEM_NAME=woa64 ^
+  -DLLVM_ENABLE_PROJECTS="clang;clang-tools-extra;flang;lld;lldb;compiler-rt;mlir"
+set cmake_flags=%all_cmake_flags:\=/%
----------------
hans wrote:
> ..and mlir?
> 
> I wonder if we should try to keep LLVM_ENABLE_PROJECTS the same across targets, maybe there could be a flag to add flang to it if we need that.
mlir + flang included to woa build since 14.0.0. IMO should be added to x64 as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142983