Added the option im_enable_openmp to compile the library with the openmp flag
What is the new feature
The library now compiles OpenMP codes even when the OpenMP flag is not added manually in CMAKE_Fortran_FLAGS
How is it implemented
Added the option IM_ENABLE_OPENMP
, checked whether OpenMP is available as a package and added the flags in the existing Fortran_FLAGS
Mandatory steps before review
-
Gitlab CI passes (Hint: use make format
for linting) -
New feature is covered by additional unit tests -
Mark the merge request as ready by removing Draft:
Mandatory steps before merge
-
Reviewed by a maintainer -
Incorporate review suggestions -
Prior to merging, please remove any boilerplate from the MR description, retaining only the Please describe your feature in a couple of words and describe important implementation details of the feature section to maintain
You are not supposed to merge this request by yourself, the maintainers of libiconmath take care of this action!
Edited by Yen-Chen Chen
Merge request reports
Activity
assigned to @k202170
added 1 commit
- 195b00d9 - fix: changed the default of enable_openmp to OFF
added 1 commit
- 91d2e344 - build: checking whether the openmp macro is already defined
added 8 commits
-
4fb7d3c3 - 1 commit from branch
main
- f70a844e - build: enabled compilation with openmp
- 5135a233 - fix: a bug was fixed in mo_lib_divrot to use openmp enabling
- fe7b92bb - ci: added a test to check openmp enabling
- 154ab3c3 - ci: fixed a bug introduced in the last commit
- 619ea3ef - ci: added an additional test to check openmp enabling for an intel compiler
- e5c56e2d - fix: changed the default of enable_openmp to OFF
- d424c62f - build: checking whether the openmp macro is already defined
Toggle commit list-
4fb7d3c3 - 1 commit from branch
added ready for review label
requested review from @b382190
added 9 commits
-
edcf8545 - 1 commit from branch
main
- 1fc65769 - build: enabled compilation with openmp
- e7e620f3 - fix: a bug was fixed in mo_lib_divrot to use openmp enabling
- bd598bb2 - ci: added a test to check openmp enabling
- f5f5b074 - ci: fixed a bug introduced in the last commit
- 8b7781fa - ci: added an additional test to check openmp enabling for an intel compiler
- 328c4151 - fix: changed the default of enable_openmp to OFF
- 2ee46826 - build: checking whether the openmp macro is already defined
- 9f4e65cd - ci: replaced the remaining levante-fake with levante
Toggle commit list-
edcf8545 - 1 commit from branch
- Resolved by Yen-Chen Chen
- Resolved by Yen-Chen Chen
- Resolved by Yen-Chen Chen
requested review from @b382190
Please register or sign in to reply