Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

Refactoring has started in
#18143
#18251
but this final step wasn't performed before due to some unittests failing. Seems not to be the case anymore.

Refactoring has started in
intel#18143
intel#18251
but this final step wasn't performed before due to some unittests
failing. Seems not to be the case anymore.
@aelovikov-intel aelovikov-intel marked this pull request as ready for review July 29, 2025 01:10
@aelovikov-intel aelovikov-intel requested a review from a team as a code owner July 29, 2025 01:11
@aelovikov-intel aelovikov-intel merged commit 34a5b8e into intel:sycl Jul 29, 2025
39 of 40 checks passed
@aelovikov-intel aelovikov-intel deleted the plt-dev-own branch July 29, 2025 17:10
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Jul 29, 2025
Similar to intel#19613.

Refactoring has started in
intel#18143
intel#18251

This didn't work back then due to some unittests failing (because
`global_handler` shutdown is different with unittests) but it doesn't
seem to be an issue any more, probably due to other refactoring PRs that
prefer raw ptr/refs over `std::shared_ptr` elsewhere.
aelovikov-intel added a commit that referenced this pull request Aug 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants