Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EXSWHTEC-106 - Reimplement tests for hipOccupancyMaxActiveBlocksPerMultiprocessor and hipOccupancyMaxPotentialBlockSize APIs #46

Merged
merged 12 commits into from
Jun 28, 2023

Conversation

nives-vukovic
Copy link
Contributor

  • Add helper file occupancy_common.hh with parameterized templates
  • Expand all positive and negative tests to use the templates

gargrahul and others added 7 commits October 26, 2022 03:59
Change-Id: I66f0c09e9c7405ec7430b1883e0e89542fdb87a0
Change-Id: I212b82b1b3a78a368b85ea64e338371a34b405f9
Change-Id: Ib455f72b5be77e1a81137d15c07ea41161b16a3e
Change-Id: Ief96e274f4143e80ceb3e40f04d38ae217777583
Change-Id: I9c03cde09b42c8e3726153c2a177359efc8d6d29
…ltiprocessor and hipOccupancyMaxPotentialBlockSize APIs

- Add helper file occupancy_common.hh with parameterized templates
- Expand all positive and negative tests to use the templates
@nives-vukovic nives-vukovic marked this pull request as ready for review December 9, 2022 14:44
},
blockSize);

#if HT_NVIDIA // EXSWHTEC-219
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this and use json file. I talked to @mangupta and we just want to avoid the whole test case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chrispaquot It has been removed and test has been disabled using json files.

@rorake
Copy link
Contributor

rorake commented Jun 21, 2023

@nives-vukovic
Will you please resolve the merge conflicts for this PR?

@nives-vukovic
Copy link
Contributor Author

@rorake Conflicts are resolved.

@mangupta mangupta merged commit dd9b9b0 into ROCm:develop Jun 28, 2023
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.

5 participants