Skip to content

Commit 3b0be29

Browse files
[SYCL] re-enable Scheduler StreamBufferDeallocation unit test (#15292)
A month ago, we had a strange intermittent failure of one of the unit tests: #15049 and in response to that we disabled the test. It is unclear why that might have been failing. But that was when there were some other disrupting changes, and fallout from that has since been fixed. I have retested this on Windows for 100,000 iterations, and not a single failure seen. Tested on the CI, ran successfully there too. I'd like to reenable this test with this PR. If the CI system starts failing on PR, we can just revert this PR in coming days.
1 parent 37d1d51 commit 3b0be29

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

sycl/unittests/scheduler/GraphCleanup.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -307,12 +307,7 @@ struct AttachSchedulerWrapper {
307307
};
308308

309309
// Check that stream buffers are released alongside graph cleanup.
310-
// https://github.com/intel/llvm/issues/15049
311-
#ifdef _WIN32
312-
TEST_F(SchedulerTest, DISABLED_StreamBufferDeallocation) {
313-
#else
314310
TEST_F(SchedulerTest, StreamBufferDeallocation) {
315-
#endif
316311
unittest::UrMock<> Mock;
317312
platform Plt = sycl::platform();
318313
context Ctx{Plt};

0 commit comments

Comments
 (0)