ROCM 5.6 release: Incorrect version in hipvars.pm
In ROCM 5.6 release, hipvars.pm is still using $HIP_BASE_VERSION_MINOR = "5"; and $HIP_BASE_VERSION_PATCH = "0";
@david-salinas is looking into this, specifically if these macros can be removed so we don't need to worry about them falling out of sync again in the future.
A couple of comments to consider for this issue:
- we are deprecating the use of Perl in HIPCC, and these Perl files will be removed in an upcoming Release.
- There is a C++ version of "hipcc" and "hipconfig" that have a similar issue.
- The HIP version information is actually kept in a config file ($HIP_PATH/share/hip/version), that the hipcc/hipconfig tools read. The use of local variables for HIP version are just place holders in case the version file cannot be read.
- We are likely not going to make any changes to hipcc in already released Releases, that don't provide technical/funcitonal benefit
We can clean up the C++ hipcc/hipconfig tools so that we no longer have to manually update local variable place holder version numbers.
This fix will be included in ROCm 6.4. Closing as resolved, please feel free to open another issue if there are further problems upon release. Thank you!