Skip to content

Change the syntax of saving result of ::omp_get_num_threads()#1677

Merged
bernhardmgruber merged 1 commit intoalpaka-group:developfrom
sbastrakov:fix-workaroundIntelInternalError
Apr 5, 2022
Merged

Change the syntax of saving result of ::omp_get_num_threads()#1677
bernhardmgruber merged 1 commit intoalpaka-group:developfrom
sbastrakov:fix-workaroundIntelInternalError

Conversation

@sbastrakov
Copy link
Copy Markdown
Member

The previously used syntax was valid but sometimes caused internal error for Intel compiler in CI:

/home/runner/work/alpaka/alpaka/include/alpaka/kernel/TaskKernelCpuOmp2Threads.hpp(134): internal error: assertion failed: cd_new_hash_table: malloc failed (cdhashtbl.c, line 134 in cd_new_hash_table)

                                  int const numThreads(::omp_get_num_threads());

The changed one suggested by @bernhardmgruber appears to work around the issue.

I faced this issue sometimes with #1336 and after making the fix in that branch it no longer happened, so I believe it indeed works around the issue.

The previously used syntax was valid but sometimes caused internal error for Intel compiler in CI.
The changed one appears to work around the issue.
@sbastrakov
Copy link
Copy Markdown
Member Author

Ready to be merged.

@bernhardmgruber bernhardmgruber merged commit 069483f into alpaka-group:develop Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants