mirror of
https://github.com/blender/blender
synced 2026-10-25 03:07:01 +03:00
There is a strong coupling between the API version an application passes to the hiprtCreateContext() and the HIP-RT library: if there is a mismatch the context will fail to be created. Additionally, the HIP-RT library does not have static dependencies as it loads HIP SDK dynamically. It all makes the wrangler be not so useful, and harmful in cases when Blender is packaged for different Linux distros which might use HIP-RT library of a different version from what Cycles is currently expecting via the bundled hiprtew.h. The HIPRT_API_VERSION is now coming from the hiprt/hiprt.h so it is guaranteed that Cycles will request the same API version as the library is was compiled with. The slightly annoying part is that this approach requires the dynamic libraries to have API version in the name of the library to function properly This is because of the SONAME which still mentions the original library name. The similar thing happens on Windows where the hiprt0200564.lib points to the hiprt0200564.dll. There might be some way to do the rename during install, but it is a bit tricky, especially due to the manifest on Windows. It might be the easiest to simply rename precompiled libraries. Pull Request: https://projects.blender.org/blender/blender/pulls/153892 |
||
|---|---|---|
| .. | ||
| hipew.h | ||