Skip to content

[SYCL][E2E] Fix memory leak in device_architecture_comparison_on_device_aot.cpp #16667

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jan 23, 2025
Merged

[SYCL][E2E] Fix memory leak in device_architecture_comparison_on_device_aot.cpp #16667

merged 2 commits into from
Jan 23, 2025

Conversation

@uditagarwal97
Copy link
Contributor

@uditagarwal97 uditagarwal97 commented Jan 16, 2025

No description provided.

@uditagarwal97 uditagarwal97 self-assigned this Jan 16, 2025
@uditagarwal97
Copy link
Contributor Author

uditagarwal97 commented Jan 16, 2025 •
edited
Loading

Note to reviewers:
The only functional change in this PR is the addition of sycl::free towards the end of the test case, rest all is NFC.

@uditagarwal97 uditagarwal97 marked this pull request as ready for review January 16, 2025 20:55
@uditagarwal97 uditagarwal97 requested a review from a team as a code owner January 16, 2025 20:55
@uditagarwal97 uditagarwal97 mentioned this pull request Jan 16, 2025
Merged
@uditagarwal97
Copy link
Contributor Author

uditagarwal97 commented Jan 17, 2025 •
edited
Loading

@intel/llvm-gatekeepers the PR is ready to be merged. CI failures are unrelated.

@uditagarwal97
Copy link
Contributor Author

uditagarwal97 commented Jan 23, 2025

@intel/llvm-gatekeepers please merge this PR.

@sarnex sarnex merged commit 9da9bc8 into intel:sycl Jan 23, 2025
17 checks passed
@uditagarwal97 uditagarwal97 deleted the fix_dev_arch_comp branch November 5, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aelovikov-intel aelovikov-intel aelovikov-intel approved these changes

Assignees

@uditagarwal97 uditagarwal97

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@uditagarwal97 @aelovikov-intel @sarnex