[nvSHMEM][E2E] Added build-only E2E for nvSHMEM APIs #913

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

Merged

zhimingwang36 merged 5 commits into oneapi-src:SYCLomatic from TejaX-Alaghari:nvSHMEM

May 27, 2025

Contributor

TejaX-Alaghari commented May 21, 2025

This PR adds build-only E2E case for below nvSHMEM APIs:

nvshmem_putmem_nbi
nvshmem_signal_wait_until
nvshmemx_signal_op
nvshmem_putmem_signal_nbi
nvshmem_malloc
nvshmem_free
nvshmemx_init_attr
nvshmem_ptr

Types:

nvshmemx_init_attr_t

Flags:

NVSHMEMX_INIT_WITH_MPI_COMM


          Added build-only E2E for nvSHMEM APIs

da5271e

TejaX-Alaghari requested a review from a team as a code owner

May 21, 2025 06:53

TejaX-Alaghari requested review from the-slow-one and intwanghao

May 21, 2025 06:53


          Modified testing config

934ff7d

tomflinda requested changes

View reviewed changes

Contributor

tomflinda left a comment

If only test build process, it can be enabled in SYCLomatic lit test, no need to add test here.+

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved

tomflinda reviewed

View reviewed changes

features/feature_case/nvSHMEM/nvshmem.cu Outdated Show resolved Hide resolved


          Removed dead code

833932b

tomflinda reviewed

View reviewed changes

features/test_feature.py Outdated Show resolved Hide resolved


          Removed quiet & barrier api comments

43469ee

tomflinda reviewed

View reviewed changes

features/test_feature.py Outdated Show resolved Hide resolved


          Removed test case from exec_tests

70c7c9b

tomflinda approved these changes

View reviewed changes

Contributor

tomflinda left a comment

LGTM

zhimingwang36 approved these changes

View reviewed changes

zhimingwang36 merged commit e168cad into oneapi-src:SYCLomatic

2 of 6 checks passed

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

Reviewers

tomflinda tomflinda approved these changes

zhimingwang36 zhimingwang36 approved these changes

the-slow-one Awaiting requested review from the-slow-one the-slow-one is a code owner automatically assigned from oneapi-src/syclomatic-reviewers

intwanghao Awaiting requested review from intwanghao intwanghao is a code owner automatically assigned from oneapi-src/syclomatic-reviewers

Labels

None yet