Skip to content

Commit

Permalink
HIP SDK 6.2
Browse files Browse the repository at this point in the history
  • Loading branch information
lshqqytiger committed Dec 18, 2024
1 parent c0804ca commit 1b6e012
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 18 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:

env:
CARGO_TERM_COLOR: always
ROCM_VERSION: "6.1.3"
ROCM_VERSION: "6.2.4"

jobs:
build_lin:
Expand Down Expand Up @@ -61,9 +61,9 @@ jobs:
- uses: actions/checkout@v4
with:
submodules: true
- name: Install AMD HIP SDK 6.1
- name: Install AMD HIP SDK 6.2
run: |
C:\msys64\usr\bin\wget.exe https://download.amd.com/developer/eula/rocm-hub/AMD-Software-PRO-Edition-24.Q3-Win10-Win11-For-HIP.exe -O "amdgpu-install.exe"
C:\msys64\usr\bin\wget.exe https://download.amd.com/developer/eula/rocm-hub/AMD-Software-PRO-Edition-24.Q4-Win10-Win11-For-HIP.exe -O "amdgpu-install.exe"
.\amdgpu-install.exe -Install -View:1
Start-Sleep -Seconds 60
$setupId = (Get-Process ATISetup).id
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:

env:
CARGO_TERM_COLOR: always
ROCM_VERSION: "6.1.3"
ROCM_VERSION: "6.2.4"

jobs:
release:
Expand Down Expand Up @@ -127,9 +127,9 @@ jobs:
- uses: actions/checkout@v4
with:
submodules: true
- name: Install AMD HIP SDK 6.1
- name: Install AMD HIP SDK 6.2
run: |
C:\msys64\usr\bin\wget.exe https://download.amd.com/developer/eula/rocm-hub/AMD-Software-PRO-Edition-24.Q3-Win10-Win11-For-HIP.exe -O "amdgpu-install.exe"
C:\msys64\usr\bin\wget.exe https://download.amd.com/developer/eula/rocm-hub/AMD-Software-PRO-Edition-24.Q4-Win10-Win11-For-HIP.exe -O "amdgpu-install.exe"
.\amdgpu-install.exe -Install -View:1
Start-Sleep -Seconds 60
$setupId = (Get-Process ATISetup).id
Expand Down
24 changes: 12 additions & 12 deletions xtask/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,10 @@ enum Subcommand {

impl Default for Subcommand {
fn default() -> Self {
Subcommand::Build(BuildCommand { release: false, rocm5: false })
Subcommand::Build(BuildCommand {
release: false,
rocm5: false,
})
}
}

Expand Down Expand Up @@ -232,23 +235,20 @@ fn build_impl(is_debug: bool, rocm5: bool) -> Result<Workspace, DynError> {
if !is_debug {
command.arg("--release");
}
if cfg!(windows) { // Should we allow ROCm 5 for Linux?
if cfg!(windows) {
// Should we allow ROCm 5 for Linux?
if rocm5 {
command.args(["--features", "rocm5"]);
}
if let Ok(path_default) = env::var("HIP_PATH") {
env::set_var("HIP_PATH",
env::var(
if rocm5 {
"HIP_PATH_57"
} else {
"HIP_PATH_61"
}
)
.unwrap_or(path_default)
env::set_var(
"HIP_PATH",
env::var(if rocm5 { "HIP_PATH_57" } else { "HIP_PATH_62" }).unwrap_or(path_default),
);
} else {
return Err("Could not find HIP SDK installed. Please check if HIP_PATH is set.".into());
return Err(
"Could not find HIP SDK installed. Please check if HIP_PATH is set.".into(),
);
}
}
let build_result = command.status()?.code().unwrap();
Expand Down

0 comments on commit 1b6e012

Please sign in to comment.