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.

Sign up for GitHub

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

Jump to bottom

Update Kernel Launch Example #3631

Open

harkgill-amd wants to merge 2 commits into docs/develop from harkgill/exampleFix

harkgill-amd commented Oct 3, 2024 •

edited

Loading

Updated the kernel launch example to fix the issues highlighted in #3428.

Example is essentially pseudo-code and isn't expected to compile but had a few obvious errors that are corrected.

hipLaunchParm was causing parameter mismatch on kernel launch
__host__ and __device__ annotations were missing on PlusOne
gid indexing updated
Inconsistent use of gridDim and groupDim fixed
wrong variable names (N vs n)


          Update Kernel Launch Example

5ba6443

harkgill-amd requested review from chrispaquot, gandryey, saleelk, mangupta and rakesroy as code owners

October 3, 2024 18:47

harkgill-amd requested a review from neon60

October 3, 2024 18:47


          Update Kernel Launch Example

a06e673

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

chrispaquot Awaiting requested review from chrispaquot chrispaquot is a code owner

gandryey Awaiting requested review from gandryey gandryey is a code owner

saleelk Awaiting requested review from saleelk saleelk is a code owner

mangupta Awaiting requested review from mangupta mangupta is a code owner

rakesroy Awaiting requested review from rakesroy rakesroy is a code owner

neon60 Awaiting requested review from neon60

Labels

None yet