[cmake] Guard another instance where symlinks are being created

[Hahnfeld]

I'm not sure about this patch: These commands do not install the aliases, they are created in the build directory and may be needed for testing.

(And shouldn't this say if (LIBOMP_INSTALL_ALIASES), according to your change in D47221?)

[azharudd]

In D47224#1112505, @Hahnfeld wrote:

I'm not sure about this patch: These commands do not install the aliases, they are created in the build directory and may be needed for testing.

This should be NFC as by default that flag would be ON. I see it being used in lines 306-314 which is dealing with installing the symlinks. Maybe this is just duplicate code.

(And shouldn't this say if (LIBOMP_INSTALL_ALIASES), according to your change in D47221?)

I just reused the existing if condition in this file and wanted to remain consistent. At least it doesn't cause a cmake error in this case as the if condition would still be correct.

[Hahnfeld]

In D47224#1118242, @azharudd wrote:

In D47224#1112505, @Hahnfeld wrote:

I'm not sure about this patch: These commands do not install the aliases, they are created in the build directory and may be needed for testing.

This should be NFC as by default that flag would be ON. I see it being used in lines 306-314 which is dealing with installing the symlinks. Maybe this is just duplicate code.

It changes things if you set a non-default patch, so it's definitely not NFC, no matter what the default is.

(And shouldn't this say if (LIBOMP_INSTALL_ALIASES), according to your change in D47221?)

I just reused the existing if condition in this file and wanted to remain consistent. At least it doesn't cause a cmake error in this case as the if condition would still be correct.

You made the point that the existing conditions are wrong, so I don't think we should add new code that doesn't follow the CMake documentation.

runtime/src/CMakeLists.txt

 if(NOT LIBOMP_LIBRARY_DIR)
   set(LIBOMP_LIBRARY_DIR ${CMAKE_CURRENT_BINARY_DIR})
   set(LIBOMP_LIBRARY_DIR ${CMAKE_CURRENT_BINARY_DIR} PARENT_SCOPE)
 else()
   set(LIBOMP_LIBRARY_DIR ${LIBOMP_LIBRARY_DIR} PARENT_SCOPE)
 endif()
 
 # Add symbolic links to libomp
+if(${LIBOMP_INSTALL_ALIASES})
   if(NOT WIN32)
     add_custom_command(TARGET omp POST_BUILD
       COMMAND ${CMAKE_COMMAND} -E create_symlink ${LIBOMP_LIB_FILE}
         libgomp${LIBOMP_LIBRARY_SUFFIX}
       COMMAND ${CMAKE_COMMAND} -E create_symlink ${LIBOMP_LIB_FILE}
         libiomp5${LIBOMP_LIBRARY_SUFFIX}
       WORKING_DIRECTORY ${LIBOMP_LIBRARY_DIR}
     )
   endif()
+endif()
 
 # Linking command will include libraries in LIBOMP_CONFIGURED_LIBFLAGS
 libomp_get_libflags(LIBOMP_CONFIGURED_LIBFLAGS)
 target_link_libraries(omp ${LIBOMP_CONFIGURED_LIBFLAGS} ${CMAKE_DL_LIBS})
 
 # Create *.inc before compiling any sources
 # objects depend on : .inc files
 add_custom_target(libomp-needed-headers DEPENDS kmp_i18n_id.inc kmp_i18n_default.inc)