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

Make KOKKOS_ALIGN_SIZE a configure-time option #1004

Closed
mhoemmen opened this issue Jul 31, 2017 · 2 comments
Closed

Make KOKKOS_ALIGN_SIZE a configure-time option #1004

mhoemmen opened this issue Jul 31, 2017 · 2 comments
Assignees
Labels
Enhancement Improve existing capability; will potentially require voting
Milestone

Comments

@mhoemmen
Copy link
Contributor

Request by @alanw0 . A user would like to use AVX instructions with explicit SIMD types, with Kokkos::View, with the GCC compiler. This requires that KOKKOS_ALIGN_SIZE be at least 32, for a vector width of 4. However, KOKKOS_ALIGN_SIZE is 64 with the Intel compiler and 16 with all other compilers. Alan would like one of the following:

  1. KOKKOS_ALIGN_SIZE should depend only on the SIMD vector width, not on the compiler.
  2. Users should be able to set KOKKOS_ALIGN_SIZE as a configure-time option.
@ibaned ibaned added the Enhancement Improve existing capability; will potentially require voting label Jul 31, 2017
@ibaned ibaned added this to the 2017-September (mid) milestone Jul 31, 2017
@alanw0
Copy link

alanw0 commented Jul 31, 2017

Thanks for lodging this issue Mark. I want to also reference Shreyas @sayerhs, so he can stay updated on this.

@hcedwar hcedwar modified the milestones: 2017 September, 2017 October Sep 13, 2017
@hcedwar hcedwar self-assigned this Sep 21, 2017
@hcedwar hcedwar modified the milestones: 2017 October, 2017 September Sep 21, 2017
@hcedwar
Copy link
Contributor

hcedwar commented Sep 21, 2017

Resolved in PR #1115
KOKKOS_MEMORY_ALIGNMENT is the single configurable memory alignment parameter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improve existing capability; will potentially require voting
Projects
None yet
Development

No branches or pull requests

5 participants