experiments/STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix for the broken --config=rocm build.

Browse Source 
Note: This ia a different PR from #28189 (same symptom, different cause and fix)

The `--config=rocm` build was broken by the following commit.

9b1b3df00e

The changes made by the above commit were missing a couple of changes for the ROCm platform, whias was leading to the build failure. Adding those changes to make the `--config=rocm` build working again.
This commit is contained in:
Deven Desai 2019-04-29 15:11:07 +00:00
parent c38b41d7c8
commit 71791470c7
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tensorflow/stream_executor/rocm/rocm_diagnostics.cc
View File

@ -180,9 +180,9 @@ port::StatusOr<DriverVersion> Diagnostician::FindDsoVersion() {
}
string dso_version = dot + strlen(so_suffix);
// TODO(b/22689637): Eliminate the explicit namespace if possible.
string stripped_dso_version = absl::StripSuffix(dso_version, ".ld64");
auto stripped_dso_version = absl::StripSuffix(dso_version, ".ld64");
auto result = static_cast<port::StatusOr<DriverVersion>*>(data);
*result = rocm::StringToDriverVersion(stripped_dso_version);
*result = rocm::StringToDriverVersion(string(stripped_dso_version));
return 1;
}
return 0;
@ -210,8 +210,8 @@ port::StatusOr<DriverVersion> Diagnostician::FindKernelModuleVersion(
size_t space_index = version_and_rest.find(" ");
auto kernel_version = version_and_rest.substr(0, space_index);
// TODO(b/22689637): Eliminate the explicit namespace if possible.
string stripped_kernel_version = absl::StripSuffix(kernel_version, ".ld64");
return rocm::StringToDriverVersion(stripped_kernel_version);
auto stripped_kernel_version = absl::StripSuffix(kernel_version, ".ld64");
return rocm::StringToDriverVersion(string(stripped_kernel_version));
}
void Diagnostician::WarnOnDsoKernelMismatch(