Skip to content

Commit b7e481e

Browse files
committed
fix: make the x86 arch optional for llvm windows
1 parent e647ab7 commit b7e481e

File tree

5 files changed

+6
-5
lines changed

5 files changed

+6
-5
lines changed

dist/legacy/setup-cpp.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/legacy/setup-cpp.js.map

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/modern/setup-cpp.mjs

+1-1
Large diffs are not rendered by default.

dist/modern/setup-cpp.mjs.map

+1-1
Large diffs are not rendered by default.

src/llvm/llvm_url.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ async function getAssetKeywords(platform: string, arch: string) {
8585
case "win32": {
8686
const osKeywordsChoice: string[] = ["windows", "Windows"]
8787
if (x86_64.includes(arch)) {
88-
osKeywordsChoice.push("win64", "win64", "win64", "x86_64", "X64")
88+
osKeywordsChoice.push("win64", "win64", "win64")
89+
optionalKeywords.push(["x86_64", "X64"])
8990
// TODO fallback to win32 if win64 is not available (e.g. for LLVM 3.6.2 and older)
9091
} else if (x86.includes(arch)) {
9192
osKeywordsChoice.push("win32")

0 commit comments

Comments
 (0)