Skip to content

First pass at implementing hierarchical algorith for barrier#1220

Draft
avinciguerra07 wants to merge 1 commit intoSandia-OpenSHMEM:mainfrom
avinciguerra07:hier
Draft

First pass at implementing hierarchical algorith for barrier#1220
avinciguerra07 wants to merge 1 commit intoSandia-OpenSHMEM:mainfrom
avinciguerra07:hier

Conversation

@avinciguerra07
Copy link

No description provided.

@bcmIntc
Copy link
Collaborator

bcmIntc commented Feb 4, 2026

It looks like we need a declaration to build, I added:


src/shmem_collectives.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/src/shmem_collectives.h b/src/shmem_collectives.h
index 1973664f..820e016a 100644
--- a/src/shmem_collectives.h
+++ b/src/shmem_collectives.h
@@ -46,6 +46,7 @@ void shmem_internal_sync_linear(int PE_start, int PE_stride, int PE_size, long *
void shmem_internal_sync_tree(int PE_start, int PE_stride, int PE_size, long *pSync);
void shmem_internal_sync_dissem(int PE_start, int PE_stride, int PE_size, long *pSync);
void shmem_internal_sync_hier(int PE_start, int PE_stride, int PE_size, long *pSync);
+int shmem_internal_collectives_post_init(void);

static inline
void

--

@avinciguerra07 avinciguerra07 marked this pull request as draft February 4, 2026 13:02
@avinciguerra07
Copy link
Author

It looks like we need a declaration to build, I added:

src/shmem_collectives.h | 1 + 1 file changed, 1 insertion(+)

diff --git a/src/shmem_collectives.h b/src/shmem_collectives.h index 1973664f..820e016a 100644 --- a/src/shmem_collectives.h +++ b/src/shmem_collectives.h @@ -46,6 +46,7 @@ void shmem_internal_sync_linear(int PE_start, int PE_stride, int PE_size, long * void shmem_internal_sync_tree(int PE_start, int PE_stride, int PE_size, long *pSync); void shmem_internal_sync_dissem(int PE_start, int PE_stride, int PE_size, long *pSync); void shmem_internal_sync_hier(int PE_start, int PE_stride, int PE_size, long *pSync); +int shmem_internal_collectives_post_init(void);

static inline void

--

Thanks for looking at this Bryan, there are still some issues with this code, I marked it as draft.

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.

2 participants