Skip to content

Commit 44e21bd

Browse files
Tidy comments and test scaffolding
1 parent 286e009 commit 44e21bd

File tree

3 files changed

+1
-6
lines changed

3 files changed

+1
-6
lines changed

.github/actions/setup_build/action.yml

+1
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ runs:
103103
if: inputs.os == 'windows' && inputs.llvm_source == 'install'
104104
shell: pwsh
105105
env:
106+
# Map inputs.llvm_version to specific download release here.
106107
LLVM_RELEASE: "${{ inputs.llvm_version == '20' && '20.1.2' || '19.1.7' }}"
107108
run: |
108109
Invoke-WebRequest https://github.com/llvm/llvm-project/releases/download/llvmorg-$env:LLVM_RELEASE/clang+llvm-$env:LLVM_RELEASE-x86_64-pc-windows-msvc.tar.xz -OutFile clang+llvm-$env:LLVM_RELEASE-x86_64-pc-windows-msvc.tar.xz

.github/workflows/run_ock_internal_tests.yml

-3
Original file line numberDiff line numberDiff line change
@@ -198,8 +198,6 @@ jobs:
198198
# Based on: mr-windows-msvc-x86_64-llvm-previous-cl3.0-offline
199199
run_windows_msvc_x86_64_llvm_latest_cl3_0_offline:
200200
runs-on: windows-2019
201-
# do not run as PR job due to not yet supporting windows install
202-
#if: contains(inputs.target_list, 'host_x86_64_windows') && !inputs.is_pull_request
203201
if: contains(inputs.target_list, 'host_x86_64_windows')
204202
steps:
205203
- name: Setup Windows llvm base
@@ -216,7 +214,6 @@ jobs:
216214
with:
217215
llvm_version: ${{ inputs.llvm_current }}
218216
os: windows
219-
#llvm_source: ${{ inputs.llvm_source == 'install' && 'cache' || inputs.llvm_source }}
220217
llvm_source: ${{ inputs.llvm_source }}
221218
github_token: ${{ secrets.GITHUB_TOKEN }}
222219

.github/workflows/run_pr_tests_caller.yml

-3
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,3 @@ jobs:
2828
llvm_source: install
2929
llvm_previous: 19
3030
llvm_current: 20
31-
target_list: '[ "host_x86_64_windows" ]'
32-
33-

0 commit comments

Comments
 (0)