Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UR][L0] Add UR_L0_LEAKS_DEBUG key #1053

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

jandres742
Copy link

@jandres742 jandres742 commented Nov 8, 2023

Use a new environment variable, UR_L0_LEAKS_DEBUG, to check for leaks in the UR L0 adapter, instead of relying on a specific value being set in UR_L0_DEBUG.

LLVM testing: intel/llvm#11820

@jandres742 jandres742 force-pushed the url0leakkey branch 5 times, most recently from f25da73 to 3da2133 Compare November 12, 2023 00:44
Use a new environment variable, UR_L0_LEAKS_DEBUG, to check
for leaks in the UR L0 adapter, instead of relying on a specific
value being set in UR_L0_DEBUG.

Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
@jandres742
Copy link
Author

@smaslov-intel : please take a look.

@jandres742 jandres742 marked this pull request as ready for review November 13, 2023 01:55
@jandres742 jandres742 requested a review from a team as a code owner November 13, 2023 01:55
@jandres742
Copy link
Author

@kbenzie : please merge when possible.
The llvm testing patch intel/llvm#11820 does more than just testing. it also changes some of the tests to make them pass, so the changes from that patch are needed when upgrading the UR version in intel/llvm.

@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Nov 14, 2023
@kbenzie kbenzie merged commit 31b654f into oneapi-src:adapters Nov 22, 2023
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Added to PR's which are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants