Skip to content

Commit 7f23a95

Browse files
committed
Cleanups last commit was working
1 parent 36c7b9e commit 7f23a95

File tree

2 files changed

+7
-42
lines changed

2 files changed

+7
-42
lines changed

.github/workflows/release-binaries.yml

Lines changed: 6 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -67,19 +67,6 @@ jobs:
6767
test-runs-on: ${{ steps.vars.outputs.build-runs-on }}
6868

6969
steps:
70-
- name: Bash Test
71-
shell: bash
72-
run: |
73-
export PATH=C:\Strawberry\c\bin\:C:\Android\android-sdk\cmake\3.18.1\bin:$PATH
74-
export PATH=/c/Strawberry/c/bin/:$PATH
75-
cmake --version
76-
which cmake
77-
/c/Android/android-sdk/cmake/3.31.5/bin/cmake.exe --version
78-
# - name: Check long paths
79-
# run: |
80-
# Get-ChildItem -Path "C:\Strawberry\" -Recurse -Include "cmake.exe"
81-
# Get-ItemProperty -Path 'HKLM:\SYSTEM\CurrentControlSet\Control\FileSystem' -Name 'LongPathsEnabled'
82-
# New-Item -Path "C:\actions-runner\llvm-project\llvm-project\build\tools\clang\stage2-instrumented-bins\tools\clang\stage2-bins\tools\mlir\tools\mlir-linalg-ods-gen\CMakeFiles\mlir-linalg-ods-yaml-gen.dir\C_\actions-runner\llvm-project\llvm-project\llvm\resources\windows_version_resource.rc.res" -ItemType File -Force
8370
# It's good practice to use setup-python, but this is also required on macos-14
8471
# due to https://github.com/actions/runner-images/issues/10385
8572
- uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
@@ -171,6 +158,11 @@ jobs:
171158
172159
build_flang="true"
173160
161+
if [ "$RUNNER_OS" = "Windows" ]; then
162+
# The build times out on Windows, so we need to disable LTO.
163+
target_cmake_flags="$target_cmake_flags -DLLVM_RELEASE_ENABLE_LTO=OFF"
164+
fi
165+
174166
echo "target-cmake-flags=$target_cmake_flags" >> $GITHUB_OUTPUT
175167
echo "build-flang=$build_flang" >> $GITHUB_OUTPUT
176168
case "${{ inputs.runs-on }}" in
@@ -208,10 +200,7 @@ jobs:
208200
if: github.repository_owner == 'llvm'
209201
runs-on: ${{ needs.prepare.outputs.build-runs-on }}
210202
steps:
211-
# - name: Check long paths
212-
# run: |
213-
# Get-ItemProperty -Path 'HKLM:\SYSTEM\CurrentControlSet\Control\FileSystem' -Name 'LongPathsEnabled'
214-
# New-Item -Path "C:\actions-runner\llvm-project\llvm-project\build\tools\clang\stage2-instrumented-bins\tools\clang\stage2-bins\tools\mlir\tools\mlir-linalg-ods-gen\CMakeFiles\mlir-linalg-ods-yaml-gen.dir\C_\actions-runner\llvm-project\llvm-project\llvm\resources\windows_version_resource.rc.res" -ItemType File -Force
203+
215204
- name: Checkout Actions
216205
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
217206
with:
@@ -263,23 +252,6 @@ jobs:
263252
cmake -G Ninja -S S:\llvm -B S:\build `
264253
${{ needs.prepare.outputs.target-cmake-flags }} `
265254
-C clang\cmake\caches\Release.cmake `
266-
-DBOOTSTRAP_LLVM_PARALLEL_LINK_JOBS=1 `
267-
-DCMAKE_OBJECT_PATH_MAX=500 `
268-
-DCMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS=1 `
269-
-DCMAKE_CXX_USE_RESPONSE_FILE_FOR_LIBRARIES=1 `
270-
-DCMAKE_CXX_RESPONSE_FILE_LINK_FLAG="@" `
271-
-DCMAKE_NINJA_FORCE_RESPONSE_FILE=1 `
272-
-DBOOTSTRAP_CMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS=1 `
273-
-DBOOTSTRAP_CMAKE_CXX_USE_RESPONSE_FILE_FOR_LIBRARIES=1 `
274-
-DBOOTSTRAP_CMAKE_CXX_RESPONSE_FILE_LINK_FLAG="@" `
275-
-DBOOTSTRAP_CMAKE_NINJA_FORCE_RESPONSE_FILE=1 `
276-
-DBOOTSTRAP_BOOTSTRAP_CMAKE_CXX_USE_RESPONSE_FILE_FOR_OBJECTS=1 `
277-
-DBOOTSTRAP_BOOTSTRAP_CMAKE_CXX_USE_RESPONSE_FILE_FOR_LIBRARIES=1 `
278-
-DBOOTSTRAP_BOOTSTRAP_CMAKE_CXX_RESPONSE_FILE_LINK_FLAG="@" `
279-
-DBOOTSTRAP_BOOTSTRAP_CMAKE_NINJA_FORCE_RESPONSE_FILE=1 `
280-
-DBOOTSTRAP_CMAKE_OBJECT_PATH_MAX=500 `
281-
-DBOOTSTRAP_BOOTSTRAP_CMAKE_OBJECT_PATH_MAX=500 `
282-
-DBOOTSTRAP_BOOTSTRAP_LLVM_BUILD_TOOLS=OFF `
283255
-DBOOTSTRAP_BOOTSTRAP_CPACK_GENERATOR=WIX `
284256
-DBOOTSTRAP_BOOTSTRAP_CPACK_PACKAGE_FILE_NAME="${{ needs.prepare.outputs.release-binary-basename }}"
285257
@@ -327,13 +299,6 @@ jobs:
327299
with:
328300
build-prefix: ${{ steps.setup-stage.outputs.build-prefix }}
329301

330-
# cat S:/build/tools/clang/stage2-instrumented-bins/tools/clang/stage2-bins/_CPack_Packages/win64/NSIS/NSISOutput.log
331-
# cat S:/build/tools/clang/stage2-instrumented-bins/tools/clang/stage2-bins/_CPack_Packages/win64/WIX/wix.log
332-
- name: Check Disk
333-
if: always()
334-
run: |
335-
Get-WmiObject -Class Win32_LogicalDisk -ComputerName LOCALHOST | ? {$_. DriveType -eq 3} | select DeviceID, {$_.Size /1GB}, {$_.FreeSpace /1GB}
336-
337302
upload-release-binaries:
338303
name: "Upload Release Binaries"
339304
needs:

clang/cmake/caches/Release.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ set (DEFAULT_RUNTIMES "compiler-rt;libcxx")
4747
if (NOT WIN32)
4848
list(APPEND DEFAULT_RUNTIMES "libcxxabi" "libunwind")
4949
endif()
50-
set(LLVM_RELEASE_ENABLE_LTO OFF CACHE STRING "")
50+
set(LLVM_RELEASE_ENABLE_LTO THIN CACHE STRING "")
5151
set(LLVM_RELEASE_ENABLE_PGO ON CACHE BOOL "")
5252
set(LLVM_RELEASE_ENABLE_RUNTIMES ${DEFAULT_RUNTIMES} CACHE STRING "")
5353
set(LLVM_RELEASE_ENABLE_PROJECTS ${DEFAULT_PROJECTS} CACHE STRING "")

0 commit comments

Comments
 (0)