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

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18860 from ROCm:ci_fix_rocm_stream_test_20241030 1b61efe
PiperOrigin-RevId: 691517601
  • Loading branch information
hsharsha authored and Google-ML-Automation committed Oct 31, 2024
1 parent bf52e39 commit 4b6d8b4
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 4b6d8b4

Please sign in to comment.