Skip to content

Commit

Permalink
fix: prefer non-msvc version of llvm for Win64
Browse files Browse the repository at this point in the history
  • Loading branch information
aminya committed Jan 12, 2025
1 parent 5f8524e commit e950a04
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 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.

2 changes: 1 addition & 1 deletion src/llvm/llvm_url.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,6 @@ 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")
// TODO fallback to win32 if win64 is not available (e.g. for LLVM 3.6.2 and older)
Expand All @@ -95,6 +94,7 @@ 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 e950a04

Please sign in to comment.