[llvm] [offload] Fix compatibility for level_zero 25.22.33944 (PR #214215)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 6 05:52:03 PDT 2026
================
@@ -177,16 +178,15 @@ class L0CmdListManagerTy {
ze_event_handle_t SignalEvent = nullptr,
uint32_t NumWaitEvents = 0,
ze_event_handle_t *WaitEvents = nullptr) {
- auto zeCommandListAppendHost = Context.zeCommandListAppendHostFunction;
- if (!zeCommandListAppendHost)
+ if (!api_helper::canCall<zeCommandListAppendHostFunction>())
----------------
blazej-smorawski wrote:
Sure, it makes sense, but I think it might be quite challenging to implement. In case we received the error we would have to go back to dlwrapper and check if fallback is available or null the function. I don't like such approach. I think that adding global table for known unsupported it not really worth if if you want to optimize time to call level zero. I think they will just do pointer lookup and return. Optimizing function that are not working is also not a good idea I think. If a user runs performance critical things he should make sure that the path he triggers is supported.
https://github.com/llvm/llvm-project/pull/214215
More information about the llvm-commits
mailing list