Skip to content

Commit

Permalink
fix: increase the gain of win64
Browse files Browse the repository at this point in the history
  • Loading branch information
aminya committed Jan 12, 2025
1 parent 6351fa6 commit 4c12ecc
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.mjs

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.mjs.map

Large diffs are not rendered by default.

5 changes: 3 additions & 2 deletions src/llvm/llvm_url.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,10 @@ async function getAssetKeywords(platform: string, arch: string) {

switch (platform) {
case "win32": {
optionalKeywords.push("windows", "Windows")
if (x86_64.includes(arch)) {
optionalKeywords.push("win64", "x86_64", "X64")
// prefer win64 keyword over x86_64 or x64
optionalKeywords.push("win64", "win64", "win64", "x86_64", "X64")
// TODO fallback to win32 if win64 is not available (e.g. for LLVM 3.6.2 and older)
} else if (x86.includes(arch)) {
keywords.push("win32")
Expand All @@ -94,7 +96,6 @@ async function getAssetKeywords(platform: string, arch: string) {
info(`Using arch ${arch} for LLVM`)
keywords.push(arch)
}
optionalKeywords.push("windows", "Windows")
break
}
case "linux": {
Expand Down

0 comments on commit 4c12ecc

Please sign in to comment.