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

CMake: FFTW Search w/ CMake Install #2059

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Jul 6, 2021

This adds support for FFTW search with OpenMP support if FFTW was installed with CMake. This is, for instance, the case with EasyBuild based installs on Juelich's JUWELS Booster cluster.

Fixes:

  • the FFTW install does NOT define CMake targets for their sub-targets, so we need to manually find and link the _omp lib
  • the library dir hint with CMake was empty because of a differing spelling (upper vs lowercase of FFTW from pkg-config check module)

Tested:

  • installed with pkg-config, SP and DP, picked up with *.pc
  • installed with CMake, SP and DP, picked up with *config.cmake
  • installed with CMake, SP and DP, picked up with *.pc

System check:

  • still need to figure our why the pkg-config module set fftw3${HFFTWp}_LIBRARY_DIRS is unset on JUWELS (but fftw3${HFFTWp}_PREFIX is set

Follow-up to #2040
Same as Hi-PACE/hipace#553

@ax3l ax3l added bug Something isn't working Performance optimization install backend: openmp Specific to OpenMP execution (CPUs) labels Jul 6, 2021
@ax3l ax3l changed the title CMake: FFTW Search w/ CMake Install [WIP] CMake: FFTW Search w/ CMake Install Jul 6, 2021
@ax3l ax3l force-pushed the fix-fftwOMPwithCMake branch 2 times, most recently from 974eacd to 56766d4 Compare July 7, 2021 23:13
@ax3l ax3l changed the title [WIP] CMake: FFTW Search w/ CMake Install CMake: FFTW Search w/ CMake Install Jul 7, 2021
This adds support for FFTW search with OpenMP support if FFTW was
installed with CMake. This is, for instance, the case with EasyBuild
based installs on Juelich's JUWELS Booster cluster.

Fixes:
- the FFTW install does NOT define CMake targets for their sub-
  targets, so we need to manually find and link the `_omp` lib
- the library dir hint with CMake was empty because of a differing
  spelling (upper vs lowercase of FFTW from pkg-config check module)

Tested:
- installed with pkg-config, SP and DP, picked up with `*.pc`
- installed with CMake, SP and DP, picked up with `*config.cmake`
- installed with CMake, SP and DP, picked up with `*.pc`
@ax3l ax3l force-pushed the fix-fftwOMPwithCMake branch from 56766d4 to 0203275 Compare July 7, 2021 23:49
@EZoni EZoni merged commit bbd1b58 into ECP-WarpX:development Jul 8, 2021
@ax3l ax3l deleted the fix-fftwOMPwithCMake branch July 10, 2021 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend: openmp Specific to OpenMP execution (CPUs) bug Something isn't working install Performance optimization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants