Skip to content

Commit

Permalink
PR #18860: [ROCm] Remove IsEmpty check for execution_order introduced…
Browse files Browse the repository at this point in the history
… in `c3d5769`

Imported from GitHub PR #18860

Copybara import of the project:

--
1b61efe by Harsha HS <Harsha.HavanurShamsundara@amd.com>:

[ROCm] Remove IsEmpty check for execution_order introduced in c3d5769

Merging this change closes #18860

COPYBARA_INTEGRATE_REVIEW=#18860 from ROCm:ci_fix_rocm_stream_test_20241030 1b61efe
PiperOrigin-RevId: 691744865
  • Loading branch information
hsharsha authored and Google-ML-Automation committed Oct 31, 2024
1 parent 03c5352 commit 19676d5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion xla/stream_executor/cuda/cuda_stream_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,6 @@ TEST_F(CudaStreamTest, WaitForOtherStream) {
}),
IsOk());

EXPECT_THAT(execution_order, IsEmpty());
EXPECT_THAT(stream1->RecordEvent(&event), IsOk());
EXPECT_THAT(stream2->BlockHostUntilDone(), IsOk());
EXPECT_THAT(execution_order,
Expand Down
1 change: 0 additions & 1 deletion xla/stream_executor/rocm/rocm_stream_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,6 @@ TEST_F(RocmStreamTest, WaitForOtherStream) {
}),
IsOk());

EXPECT_THAT(execution_order, IsEmpty());
EXPECT_THAT(stream1->RecordEvent(&event), IsOk());
EXPECT_THAT(stream2->BlockHostUntilDone(), IsOk());
EXPECT_THAT(execution_order,
Expand Down

0 comments on commit 19676d5

Please sign in to comment.