[OPENCL] Add UR handles to OPENCL adapter

[omarahmed1111]

Redesign Opencl adapter by adding UR handles and get rid of the casting of the UR handles to OPENCL handles.

Intel/llvm testing

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.43%. Comparing base (78ef1ca) to head (97dbfd0).
Report is 199 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1176      +/-   ##
==========================================
- Coverage   14.82%   12.43%   -2.40%     
==========================================
  Files         250      241       -9     
  Lines       36220    36242      +22     
  Branches     4094     4111      +17     
==========================================
- Hits         5369     4506     -863     
- Misses      30800    31732     +932     
+ Partials       51        4      -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[omarahmed1111]

I've not reviewed the whole PR yet. I think we need to run the CTS with the Address Sanitizer enabled to make sure we're not introducing memory leaks.

Used Address sanitizer to check for memory leaks, fixed some in the UR layer here. Some are generated from OpenCL sources so left that, others are generated from UR CTS tests that will be fixed in this PR

[aarongreig]

LGTM pending open discussions. This is an important change and I bet it wasn't fun to implement, thanks @omarahmed1111

[aarongreig]

I think for native handles we should mirror l0's behaviour, that's where isNativeHandleOwned was first implemented so we can assume it reflects the original intention. The UR handles don't retain the native handle, and isNativeHandleOwned decides whether UR will release the native handle when the UR handle is released.

So for example l0 context tracks whether it owns its native handle (the param gets passed from here to here) and only uses that information when it comes to releasing the context here

[aarongreig]

here's a draft for the new spec around native handles #2193

[omarahmed1111]

That make sense, I changed it to mimic l0 behaviour.