Skip to content

Conversation

@Alexandr-Konovalov
Copy link
Contributor

Add loop to the test to check effect of kernel caching on ur(Kernel|Program)(Retain|Release) calls.

Add loop to the test to check effect of kernel caching on
ur(Kernel|Program)(Retain|Release) calls.
@Alexandr-Konovalov
Copy link
Contributor Author

Failed Tests (1):
SYCL :: Reduction/reduction_nd_lambda.cpp

Unrelated and reported in [CI][Windows][GEN12] Reduction/reduction_nd_lambda.cpp failed #19354

@Alexandr-Konovalov Alexandr-Konovalov marked this pull request as ready for review August 19, 2025 10:03
@Alexandr-Konovalov Alexandr-Konovalov requested a review from a team as a code owner August 19, 2025 10:03
Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea!

@Alexandr-Konovalov
Copy link
Contributor Author

@intel/llvm-gatekeepers, could you please look at this?

@steffenlarsen
Copy link
Contributor

@intel/llvm-gatekeepers, could you please look at this?

@Alexandr-Konovalov - It is the responsibility of the author to either:

  1. Address CI failures
  2. or mention (and open if there aren't any existing) trackers/issues regarding CI failures, if they are indeed unrelated.

@Alexandr-Konovalov
Copy link
Contributor Author

@intel/llvm-gatekeepers, could you please look at this?

@Alexandr-Konovalov - It is the responsibility of the author to either:

  1. Address CI failures
  2. or mention (and open if there aren't any existing) trackers/issues regarding CI failures, if they are indeed unrelated.

Yes, and I mention it before making the PR ready for review, please see above.

@steffenlarsen
Copy link
Contributor

Ah, my bad!

@steffenlarsen steffenlarsen merged commit e784b9e into intel:sycl Aug 20, 2025
26 of 27 checks passed
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