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

Intel debug build fails #546

Closed
ewquon opened this issue Sep 17, 2020 · 1 comment
Closed

Intel debug build fails #546

ewquon opened this issue Sep 17, 2020 · 1 comment

Comments

@ewquon
Copy link
Contributor

ewquon commented Sep 17, 2020

Bug description

[  7%] Building Fortran object modules/nwtc-library/CMakeFiles/nwtclibs.dir/src/NetLib/slatec/NWTC_SLATEC.f90.o
ifort: command line error: Unrecognized keyword 'recursion' for option '-assume'
make[2]: *** [modules/nwtc-library/CMakeFiles/nwtclibs.dir/src/NetLib/slatec/NWTC_SLATEC.f90.o] Error 1
make[1]: *** [modules/nwtc-library/CMakeFiles/nwtclibs.dir/all] Error 2
make: *** [all] Error 2

To Reproduce
Steps to reproduce the behavior in linux:

  1. Configure with -DCMAKE_BUILD_TYPE:STRING=Debug
  2. Compile with Intel(R) Fortran Intel(R) 64 Compiler for applications running on Intel(R) 64, Version 18.0.5.274 Build 20180823

Expected behavior
That the code compiles.

OpenFAST Version
dev branch, commit 103130b

Additional context
To workaround:

diff --git a/modules/nwtc-library/CMakeLists.txt b/modules/nwtc-library/CMakeLists.txt
index a32fbc3..fde58d3 100644
--- a/modules/nwtc-library/CMakeLists.txt
+++ b/modules/nwtc-library/CMakeLists.txt
@@ -67,9 +67,9 @@ if (CMAKE_BUILD_TYPE MATCHES Debug)
          set_source_files_properties(src/NetLib/slatec/dqk61.f         PROPERTIES COMPILE_FLAGS "/assume:recursion")
          set_source_files_properties(src/NetLib/slatec/qk61.f          PROPERTIES COMPILE_FLAGS "/assume:recursion")
       else()
-         set_source_files_properties(src/NetLib/slatec/NWTC_SLATEC.f90 PROPERTIES COMPILE_FLAGS "-assume recursion")
-         set_source_files_properties(src/NetLib/slatec/dqk61.f         PROPERTIES COMPILE_FLAGS "-assume recursion")
-         set_source_files_properties(src/NetLib/slatec/qk61.f          PROPERTIES COMPILE_FLAGS "-assume recursion")
+         set_source_files_properties(src/NetLib/slatec/NWTC_SLATEC.f90 PROPERTIES COMPILE_FLAGS "-recursive")
+         set_source_files_properties(src/NetLib/slatec/dqk61.f         PROPERTIES COMPILE_FLAGS "-recursive")
+         set_source_files_properties(src/NetLib/slatec/qk61.f          PROPERTIES COMPILE_FLAGS "-recursive")
       endif()
    endif()
 endif()
andrew-platt added a commit to andrew-platt/openfast that referenced this issue Sep 17, 2020
Incorrect CMake flag was specified
andrew-platt added a commit that referenced this issue Sep 23, 2020
- closes #546 
- Incorrect CMake flag was specified
- intel compiler version logic check for assume:recursion flags
@andrew-platt
Copy link
Collaborator

Fixed in #547

andrew-platt added a commit to andrew-platt/openfast that referenced this issue Oct 20, 2020
- closes OpenFAST#546 
- Incorrect CMake flag was specified
- intel compiler version logic check for assume:recursion flags
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants