llvm icon indicating copy to clipboard operation
llvm copied to clipboard

Draft: [SYCL] Fixes memory dependency leaks caused by failed kernel enqueue

Open steffenlarsen opened this issue 2 years ago • 3 comments

If a kernel enqueue fails the runtime will immediately try and clean it up. However, if it has any dependencies or users the cleanup will be skipped. This can cause the dependencies to stay alive and leak. These changes forces a full sub-graph cleanup of the command if enqueuing failed. Additionally, sub-graph cleanup is changed to account for failed kernel enqueues and will remove the failed command from its leaves.

steffenlarsen avatar Mar 11 '22 11:03 steffenlarsen

This is a reopening of https://github.com/intel/llvm/pull/5120 intended to isolate the failures caused by it.

steffenlarsen avatar Mar 11 '22 11:03 steffenlarsen

/summary:run

steffenlarsen avatar Mar 11 '22 11:03 steffenlarsen

/summary:run

bso-intel avatar Jul 07 '22 18:07 bso-intel

Closing this in favor of https://github.com/intel/llvm/pull/7594.

steffenlarsen avatar Dec 09 '22 11:12 steffenlarsen