Skip to content

EXSWHTEC-74 - Implement tests for hipMemcpy and derivatives#2992

Open
music-dino wants to merge 55 commits into
ROCm:develop_oldfrom
mirza-halilcevic:hipMemcpy_tests
Open

EXSWHTEC-74 - Implement tests for hipMemcpy and derivatives#2992
music-dino wants to merge 55 commits into
ROCm:develop_oldfrom
mirza-halilcevic:hipMemcpy_tests

Conversation

@music-dino

Copy link
Copy Markdown
Contributor

Implement positive and negative unit tests for the following APIs:

  • hipMemcpy
  • hipMemcpyWithStream
  • hipMemcpyDtoH
  • hipMemcpyHtoD
  • hipMemcpyDtoD

music-dino and others added 30 commits September 30, 2022 02:38
- Implement common case tests for all variants of hipMemcpy and
hipMemcpyAsync, and hipMemcpyWithStream
- Implement tests for hipMemcpy
- Implement tests for hipMemcpyHtoD
- Implement tests for hipMemcpyDtoH
- Implement tests for hipMemcpyDtoD
chrispaquot
chrispaquot previously approved these changes Oct 27, 2022
@music-dino music-dino marked this pull request as ready for review November 2, 2022 09:53
mangupta
mangupta previously approved these changes Nov 3, 2022
@mangupta

mangupta commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

@music-dino : Some tests fail on CI. Details shared separately.

@mangupta mangupta added the pr:needs_updates PR initially approved. But needs rework label Nov 4, 2022
@music-dino

Copy link
Copy Markdown
Contributor Author

@music-dino : Some tests fail on CI. Details shared separately.

PR updated, please proceed with merge.

@mangupta mangupta added pr:ready_for_ci PR ready to be tested and removed pr:needs_updates PR initially approved. But needs rework labels Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:ready_for_ci PR ready to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants