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.

Sign up for GitHub

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

Jump to bottom

cmake: cleanup outdated code #2349

Merged
merged 6 commits into from
Nov 13, 2023
Merged

cmake: cleanup outdated code #2349

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
7d333e0
cmake: cleanup outdated code
apwojcik Aug 31, 2023
4d93104
Merge branch 'develop' into cmake_cleanup
apwojcik Sep 18, 2023
d6910de
Merge branch 'develop' into cmake_cleanup
apwojcik Oct 9, 2023
c408599
Merge branch 'develop' into cmake_cleanup
junliume Oct 20, 2023
515fd22
Merge branch 'develop' into cmake_cleanup
junliume Nov 8, 2023
bc244c8
Merge branch 'develop' into cmake_cleanup
junliume Nov 13, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -688,7 +688,6 @@ enable_clang_tidy(
"\.hpp$"
EXTRA_ARGS
-DMIOPEN_USE_CLANG_TIDY

)
include(CppCheck)
enable_cppcheck(
Expand Down Expand Up @@ -787,5 +786,5 @@ add_subdirectory(test)
add_subdirectory(speedtests)
add_subdirectory(utils)
if(MIOPEN_ENABLE_FIN)
add_subdirectory(fin)
add_subdirectory(fin)
endif()
22 changes: 3 additions & 19 deletions src/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ configure_file("${PROJECT_SOURCE_DIR}/include/miopen/config.h.in" "${PROJECT_BIN
# configure a header file to pass the CMake version settings to the source, and package the header files in the output archive
configure_file( "${PROJECT_SOURCE_DIR}/include/miopen/version.h.in" "${PROJECT_BINARY_DIR}/include/miopen/version.h" )


if(BUILD_FILE_REORG_BACKWARD_COMPATIBILITY)
#Copy header to Build Dir for generating backward compatibility
configure_file( "${PROJECT_SOURCE_DIR}/include/miopen/miopen.h" "${PROJECT_BINARY_DIR}/include/miopen/miopen.h" )
Expand All @@ -64,16 +63,11 @@ message( STATUS "CMAKE_BUILD_TYPE= ${CMAKE_BUILD_TYPE}" )
# This is incremented when the ABI to the library changes
set( MIOpen_SOVERSION 1.0 )


function(add_kernels FILE_NAME VAR_PREFIX VAR_SUFFIX KERNEL_FILES)
set(INIT_KERNELS_LIST)
set(KERNELS_DECLS)
foreach(KERNEL_FILE ${KERNEL_FILES})
if("${CMAKE_VERSION}" VERSION_LESS 3.0)
configure_file(${KERNEL_FILE} ${KERNEL_FILE}.delete)
else()
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${KERNEL_FILE})
endif()
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${KERNEL_FILE})
get_filename_component(KERNEL_FILENAME ${KERNEL_FILE} NAME)
get_filename_component(BASE_NAME ${KERNEL_FILE} NAME_WE)
string(TOUPPER "${BASE_NAME}" KEY_NAME)
Expand Down Expand Up @@ -692,23 +686,13 @@ endif()

function(target_internal_library TARGET)
target_link_libraries(${TARGET} PRIVATE ${ARGN})
set(PASS_ARGS debug optimized)
set(DEPS)
foreach(DEP ${ARGN})
if(DEP IN_LIST PASS_ARGS)
list(APPEND DEPS ${DEP})
else()
list(APPEND DEPS $<BUILD_INTERFACE:${DEP}>)
endif()
endforeach()
target_link_libraries(${TARGET} INTERFACE ${DEPS})
Comment on lines -695 to -704
Copy link
Contributor

@atamazov atamazov Sep 10, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

@pfultz2 Do we no longer need this patch for "debug" and "optimized"? I am not quite confident with this stuff.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ping @pfultz2 and @JehandadKhan

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ping @pfultz2 and @JehandadKhan for review

Copy link
Contributor

Choose a reason for hiding this comment

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

No, it isnt used anymore. I think this was used before we could use boost cmake targets.

target_link_libraries(${TARGET} INTERFACE $<BUILD_INTERFACE:${ARGN}>)
endfunction()

target_include_directories(MIOpen PUBLIC
$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/src/include>
)

set(MIOPEN_CK_LINK_FLAGS)
if(MIOPEN_USE_COMPOSABLEKERNEL)
set(MIOPEN_CK_LINK_FLAGS composable_kernel::device_operations hip::host)
endif()
Expand Down Expand Up @@ -840,7 +824,7 @@ endif()
if(NOT WIN32 AND NOT APPLE)
find_library(LIBRT rt)
if(LIBRT)
MESSAGE(STATUS "Librt: " ${LIBRT})
message(STATUS "Librt: " ${LIBRT})
target_internal_library(MIOpen ${LIBRT})
endif()
endif()
Expand Down
1 change: 0 additions & 1 deletion src/composable_kernel/host/host_tensor/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ target_include_directories(host_tensor SYSTEM PUBLIC $<BUILD_INTERFACE:${HALF_IN
target_link_libraries(host_tensor PRIVATE hip::device)
target_link_libraries(host_tensor INTERFACE hip::host)

target_compile_features(host_tensor PUBLIC)
set_target_properties(host_tensor PROPERTIES POSITION_INDEPENDENT_CODE ON)

install(TARGETS host_tensor LIBRARY DESTINATION lib)