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

Define KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION when requesting Kokkos_ENABLE_AGGRESSIVE_VECTORIZATION=ON #4107

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

masterleinad
Copy link
Contributor

@masterleinad masterleinad commented Jun 21, 2021

KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION was replaced by KOKKOS_AGGRESSIVE_VECTORIZATION in c6844d1 but it seems that we accidentally deleted the macro definition in 0591cf565a (and rather only defined the older name).

@masterleinad masterleinad changed the title Define KOKKOS_AGGRESSIVE_VECTORIZATION when requesting Kokkos_AGGRESSIVE_VECTORIZATION=ON Define KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION when requesting Kokkos_ENABLE_AGGRESSIVE_VECTORIZATION=ON Jun 21, 2021
Copy link
Member

@dalg24 dalg24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Considering

git grep -n KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION
Makefile.kokkos:629:  tmp := $(call kokkos_append_header,"$H""define KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION")
cmake/KokkosCore_config.h.in:56:#cmakedefine KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION
cmake/kokkos_enable_options.cmake:113:  SET(KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION ON)

I am leaning towards just removing.

@arnodu
Copy link

arnodu commented Jun 21, 2021

#pragma ivdep is correctly applied to my loop when using Intel 19, the Kokkos::OpenMP backend and with the Kokkos_ENABLE_AGGRESSIVE_VECTORIZATION option in cmake now, thanks.

@dalg24
Copy link
Member

dalg24 commented Jun 21, 2021

Considering

git grep -n KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION
Makefile.kokkos:629:  tmp := $(call kokkos_append_header,"$H""define KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION")
cmake/KokkosCore_config.h.in:56:#cmakedefine KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION
cmake/kokkos_enable_options.cmake:113:  SET(KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION ON)

I am leaning towards just removing.

I take it back

git grep -n VECTORIZATION
BUILD.md:138:* Kokkos_ENABLE_AGGRESSIVE_VECTORIZATION
Makefile.kokkos:73:KOKKOS_INTERNAL_OPT_RANGE_AGGRESSIVE_VECTORIZATION := $(call kokkos_has_string,$(KOKKOS_OPTIONS),aggressive_vectorization)
Makefile.kokkos:628:ifeq ($(KOKKOS_INTERNAL_OPT_RANGE_AGGRESSIVE_VECTORIZATION), 1)
Makefile.kokkos:629:  tmp := $(call kokkos_append_header,"$H""define KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION")
cmake/KokkosCore_config.h.in:56:#cmakedefine KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION
cmake/kokkos_enable_options.cmake:51:KOKKOS_ENABLE_OPTION(AGGRESSIVE_VECTORIZATION OFF "Whether to aggressively vectorize loops")
cmake/kokkos_enable_options.cmake:112:IF (KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION)
cmake/kokkos_enable_options.cmake:113:  SET(KOKKOS_OPT_RANGE_AGGRESSIVE_VECTORIZATION ON)
core/cmake/KokkosCore_config.h.in:32:#cmakedefine KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION
core/src/Cuda/Kokkos_Cuda_Vectorization.hpp:44:#ifndef KOKKOS_CUDA_VECTORIZATION_HPP
core/src/Cuda/Kokkos_Cuda_Vectorization.hpp:45:#define KOKKOS_CUDA_VECTORIZATION_HPP
core/src/Cuda/Kokkos_Cuda_Vectorization.hpp:232:#endif  // !defined( KOKKOS_CUDA_VECTORIZATION_HPP )
core/src/HIP/Kokkos_HIP_Vectorization.hpp:45:#ifndef KOKKOS_HIP_VECTORIZATION_HPP
core/src/HIP/Kokkos_HIP_Vectorization.hpp:46:#define KOKKOS_HIP_VECTORIZATION_HPP
core/src/Kokkos_Vectorization.hpp:47:#ifndef KOKKOS_VECTORIZATION_HPP
core/src/Kokkos_Vectorization.hpp:48:#define KOKKOS_VECTORIZATION_HPP
core/src/OpenMP/Kokkos_OpenMP_Parallel.hpp:80:#ifdef KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION
core/src/OpenMP/Kokkos_OpenMP_Parallel.hpp:96:#ifdef KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION
core/src/OpenMP/Kokkos_OpenMP_Parallel.hpp:175:#ifdef KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION
core/src/Threads/Kokkos_Threads_Parallel.hpp:83:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:98:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:190:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:395:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:410:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:580:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:867:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:882:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:945:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/Threads/Kokkos_Threads_Parallel.hpp:960:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \
core/src/impl/KokkosExp_Host_IterateTile.hpp:49:#if defined(KOKKOS_ENABLE_AGGRESSIVE_VECTORIZATION) && \

Co-authored-by: Damien L-G <dalg24+github@gmail.com>
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.

None yet

5 participants