Skip to content

Commit e5bb3da

Browse files
author
Karim Vadsariya
committed
Experimental changes to remove the IHV SDK dependencies
when building onnxruntime.dll onnxruntime_shared.dll
1 parent 0273bf5 commit e5bb3da

File tree

6 files changed

+68
-51
lines changed

6 files changed

+68
-51
lines changed

cmake/CMakeLists.txt

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -634,7 +634,7 @@ if (WIN32)
634634
# structure was padded due to __declspec(align())
635635
list(APPEND ORT_WARNING_FLAGS "/wd4324")
636636
# warning C4800: Implicit conversion from 'X' to bool. Possible information loss
637-
if (onnxruntime_USE_OPENVINO)
637+
if (onnxruntime_USE_OPENVINO) #TODO[Karim] applys to all projects...
638638
list(APPEND ORT_WARNING_FLAGS "/wd4800")
639639
endif()
640640
# operator 'operator-name': deprecated between enumerations of different types
@@ -758,7 +758,7 @@ set(ONNXRUNTIME_PROVIDER_NAMES cpu)
758758
set(ORT_PROVIDER_FLAGS)
759759
set(ORT_PROVIDER_CMAKE_FLAGS)
760760

761-
if (onnxruntime_USE_CUDA)
761+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
762762
enable_language(CUDA)
763763
message( STATUS "CMAKE_CUDA_COMPILER_VERSION: ${CMAKE_CUDA_COMPILER_VERSION}")
764764

@@ -793,7 +793,7 @@ endif()
793793

794794
if (onnxruntime_USE_CUDA)
795795
list(APPEND ORT_PROVIDER_FLAGS -DUSE_CUDA=1)
796-
list(APPEND ORT_PROVIDER_CMAKE_FLAGS -Donnxruntime_USE_CUDA=1)
796+
list(APPEND ORT_PROVIDER_CMAKE_FLAGS -Donnxruntime_USE_CUDA=1) #TODO[Karim] Not used anywhere
797797
list(APPEND ONNXRUNTIME_PROVIDER_NAMES cuda)
798798

799799
if (onnxruntime_USE_FLASH_ATTENTION)
@@ -856,7 +856,7 @@ if (onnxruntime_USE_JSEP)
856856
list(APPEND ORT_PROVIDER_CMAKE_FLAGS -Donnxruntime_USE_JSEP=1)
857857
list(APPEND ONNXRUNTIME_PROVIDER_NAMES js)
858858
endif()
859-
if (onnxruntime_USE_QNN)
859+
if (onnxruntime_USE_QNN) #TODO[Karim]
860860
list(APPEND ORT_PROVIDER_FLAGS -DUSE_QNN=1)
861861
list(APPEND ORT_PROVIDER_CMAKE_FLAGS -Donnxruntime_USE_QNN=1)
862862
list(APPEND ONNXRUNTIME_PROVIDER_NAMES qnn)
@@ -1032,14 +1032,15 @@ function(onnxruntime_set_compile_flags target_name)
10321032
endif()
10331033

10341034
set_target_properties(${target_name} PROPERTIES COMPILE_WARNING_AS_ERROR ON)
1035-
if (onnxruntime_USE_CUDA)
1035+
1036+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
10361037
# Suppress a "conversion_function_not_usable" warning in gsl/span
10371038
target_compile_options(${target_name} PRIVATE "$<$<COMPILE_LANGUAGE:CUDA>:SHELL:-Xcudafe \"--diag_suppress=conversion_function_not_usable\">")
10381039
target_compile_definitions(${target_name} PRIVATE -DDISABLE_CUSPARSE_DEPRECATED)
10391040
endif()
10401041
if (MSVC)
10411042
foreach(CMAKE_CUDA_TOOLKIT_INCLUDE_DIRECTORY ${CMAKE_CUDA_TOOLKIT_INCLUDE_DIRECTORIES})
1042-
target_compile_options(${target_name} PRIVATE "$<$<COMPILE_LANGUAGE:CXX,C>:/external:I${CMAKE_CUDA_TOOLKIT_INCLUDE_DIRECTORY}>")
1043+
#target_compile_options(${target_name} PRIVATE "$<$<COMPILE_LANGUAGE:CXX,C>:/external:I${CMAKE_CUDA_TOOLKIT_INCLUDE_DIRECTORY}>")
10431044
endforeach()
10441045

10451046
foreach(onnxruntime_external_lib IN LISTS onnxruntime_EXTERNAL_LIBRARIES)
@@ -1097,7 +1098,7 @@ function(onnxruntime_set_compile_flags target_name)
10971098
target_compile_options(${target_name} PRIVATE "-Wno-unused-function")
10981099
endif()
10991100
endif()
1100-
foreach(ORT_FLAG ${ORT_PROVIDER_FLAGS})
1101+
foreach(ORT_FLAG ${ORT_PROVIDER_FLAGS}) #TODO[Karim] applies to all projects...
11011102
target_compile_definitions(${target_name} PRIVATE ${ORT_FLAG})
11021103
endforeach()
11031104
if (HAS_DEPRECATED_COPY)
@@ -1110,7 +1111,7 @@ function(onnxruntime_set_compile_flags target_name)
11101111
if("${CMAKE_C_COMPILER_ID}" STREQUAL "GNU" AND CMAKE_C_COMPILER_VERSION VERSION_LESS 13 AND CMAKE_C_COMPILER_VERSION VERSION_GREATER_EQUAL 12)
11111112
target_compile_options(${target_name} PRIVATE "$<$<COMPILE_LANGUAGE:CXX>:-Wno-maybe-uninitialized>")
11121113
endif()
1113-
if (onnxruntime_USE_CUDA)
1114+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
11141115
foreach(FLAG ${ORT_WARNING_FLAGS})
11151116
target_compile_options(${target_name} PRIVATE "$<$<COMPILE_LANGUAGE:CUDA>:SHELL:--compiler-options ${FLAG}>")
11161117
endforeach()
@@ -1332,7 +1333,7 @@ if (onnxruntime_ENABLE_TRAINING_APIS)
13321333
)
13331334
endif()
13341335

1335-
if (onnxruntime_USE_OPENVINO)
1336+
if (onnxruntime_USE_OPENVINO) #TODO[Karim] applies to both onnxruntime.dll and onnxruntime_shared.dll
13361337

13371338
add_definitions(-DUSE_OPENVINO=1)
13381339

@@ -1389,7 +1390,7 @@ if (onnxruntime_USE_OPENVINO)
13891390

13901391
endif()
13911392

1392-
if (onnxruntime_USE_VITISAI)
1393+
if (onnxruntime_USE_VITISAI) #TODO[Karim]
13931394
set(CMAKE_MODULE_PATH "${CMAKE_MODULE_PATH};${CMAKE_CURRENT_LIST_DIR}")
13941395
endif()
13951396

@@ -1411,7 +1412,7 @@ string(APPEND ORT_BUILD_INFO ", cmake cxx flags: ${CMAKE_CXX_FLAGS}")
14111412
configure_file(onnxruntime_config.h.in ${CMAKE_CURRENT_BINARY_DIR}/onnxruntime_config.h)
14121413
get_property(onnxruntime_GENERATOR_IS_MULTI_CONFIG GLOBAL PROPERTY GENERATOR_IS_MULTI_CONFIG)
14131414

1414-
if (onnxruntime_USE_CUDA)
1415+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
14151416
set(CMAKE_CUDA_RUNTIME_LIBRARY Shared)
14161417
set(CMAKE_CUDA_STANDARD 17)
14171418
if(onnxruntime_CUDA_HOME)
@@ -1685,17 +1686,17 @@ if (onnxruntime_BUILD_SHARED_LIB OR onnxruntime_BUILD_APPLE_FRAMEWORK)
16851686
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime)
16861687
endif()
16871688

1688-
if (onnxruntime_BUILD_JAVA)
1689+
if (onnxruntime_BUILD_JAVA) #TODO[Karim] *cmake files included below uses onnxruntime_USE_CUDA, USE_QNN etc variables,
16891690
message(STATUS "Java Build is enabled")
16901691
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime_java)
16911692
endif()
16921693

1693-
if (onnxruntime_BUILD_NODEJS)
1694+
if (onnxruntime_BUILD_NODEJS) #TODO[Karim]
16941695
message(STATUS "Node.js Build is enabled")
16951696
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime_nodejs)
16961697
endif()
16971698

1698-
if (onnxruntime_ENABLE_PYTHON)
1699+
if (onnxruntime_ENABLE_PYTHON) #TODO[Karim]
16991700
message(STATUS "Python Build is enabled")
17001701
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime_python)
17011702
endif()
@@ -1705,7 +1706,7 @@ if (onnxruntime_BUILD_OBJC)
17051706
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime_objectivec)
17061707
endif()
17071708

1708-
if (onnxruntime_BUILD_UNIT_TESTS)
1709+
if (onnxruntime_BUILD_UNIT_TESTS) #TODO[Karim]
17091710
list(APPEND ONNXRUNTIME_CMAKE_FILES onnxruntime_unittests)
17101711
endif()
17111712

@@ -1753,7 +1754,7 @@ if (WIN32 AND NOT GDK_PLATFORM AND NOT CMAKE_CROSSCOMPILING)
17531754
endif()
17541755

17551756
foreach(target_name ${ONNXRUNTIME_CMAKE_FILES})
1756-
include(${target_name}.cmake)
1757+
include(${target_name}.cmake) #TODO[Karim]
17571758
endforeach()
17581759
if (UNIX)
17591760
option(BUILD_PKGCONFIG_FILES "Build and install pkg-config files" ON)

cmake/external/onnxruntime_external_deps.cmake

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -400,7 +400,7 @@ if ((CPUINFO_SUPPORTED OR onnxruntime_USE_XNNPACK) AND NOT ANDROID)
400400
endif()
401401
endif()
402402

403-
if(onnxruntime_USE_CUDA)
403+
if(onnxruntime_USE_CUDA) #TODO[Karim]
404404
FetchContent_Declare(
405405
GSL
406406
URL ${DEP_URL_microsoft_gsl}
@@ -682,7 +682,8 @@ if (onnxruntime_USE_WEBGPU)
682682
endif()
683683

684684
set(onnxruntime_LINK_DIRS)
685-
if (onnxruntime_USE_CUDA)
685+
686+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
686687
find_package(CUDAToolkit REQUIRED)
687688

688689
if(onnxruntime_CUDNN_HOME)

cmake/onnxruntime.cmake

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ function(get_c_cxx_api_headers HEADERS_VAR)
3737
endif()
3838

3939
# need to add header files for enabled EPs
40-
foreach(f ${ONNXRUNTIME_PROVIDER_NAMES})
40+
foreach(f ${ONNXRUNTIME_PROVIDER_NAMES}) #TODO[Karim]
4141
# The header files in include/onnxruntime/core/providers/cuda directory cannot be flattened to the same directory
4242
# with onnxruntime_c_api.h . Most other EPs probably also do not work in this way.
4343
if((NOT f STREQUAL cuda) AND (NOT f STREQUAL rocm))
@@ -66,12 +66,12 @@ if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "AIX")
6666
add_custom_command(OUTPUT ${SYMBOL_FILE} ${CMAKE_CURRENT_BINARY_DIR}/generated_source.c
6767
COMMAND ${Python_EXECUTABLE} "${REPO_ROOT}/tools/ci_build/gen_def.py"
6868
--version_file "${ONNXRUNTIME_ROOT}/../VERSION_NUMBER" --src_root "${ONNXRUNTIME_ROOT}"
69-
--config ${ONNXRUNTIME_PROVIDER_NAMES} --style=${OUTPUT_STYLE} --output ${SYMBOL_FILE}
69+
--config ${ONNXRUNTIME_PROVIDER_NAMES} --style=${OUTPUT_STYLE} --output ${SYMBOL_FILE} #TODO[Karim]
7070
--output_source ${CMAKE_CURRENT_BINARY_DIR}/generated_source.c
7171
DEPENDS ${SYMBOL_FILES}
7272
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
7373

74-
add_custom_target(onnxruntime_generate_def ALL DEPENDS ${SYMBOL_FILE} ${CMAKE_CURRENT_BINARY_DIR}/generated_source.c)
74+
add_custom_target(onnxruntime_generate_def ALL DEPENDS ${SYMBOL_FILE} ${CMAKE_CURRENT_BINARY_DIR}/generated_source.c) #TODO[Karim]
7575
endif()
7676
if(WIN32)
7777
onnxruntime_add_shared_library(onnxruntime
@@ -208,7 +208,7 @@ set(onnxruntime_INTERNAL_LIBRARIES
208208
${PROVIDERS_COREML}
209209
${PROVIDERS_DML}
210210
${PROVIDERS_NNAPI}
211-
${PROVIDERS_QNN}
211+
${PROVIDERS_QNN} #TODO[Karim] ??
212212
${PROVIDERS_SNPE}
213213
${PROVIDERS_RKNPU}
214214
${PROVIDERS_VSINPU}

cmake/onnxruntime_providers.cmake

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,10 @@ function(add_op_reduction_include_dirs target)
5959
endfunction()
6060

6161

62-
if(onnxruntime_USE_VITISAI)
62+
if(onnxruntime_USE_VITISAI) #TODO[Karim] seems like not used??
6363
set(PROVIDERS_VITISAI onnxruntime_providers_vitisai)
6464
endif()
65-
if(onnxruntime_USE_CUDA)
65+
if(onnxruntime_USE_CUDA) #TODO[Karim]
6666
set(PROVIDERS_CUDA onnxruntime_providers_cuda)
6767
endif()
6868
if(onnxruntime_USE_COREML)
@@ -74,7 +74,7 @@ endif()
7474
if(onnxruntime_USE_JSEP)
7575
set(PROVIDERS_JS onnxruntime_providers_js)
7676
endif()
77-
if(onnxruntime_USE_QNN)
77+
if(onnxruntime_USE_QNN) #TODO[Karim]
7878
set(PROVIDERS_QNN onnxruntime_providers_qnn)
7979
endif()
8080
if(onnxruntime_USE_RKNPU)
@@ -123,23 +123,23 @@ if(onnxruntime_USE_SNPE)
123123
endif()
124124

125125
include(onnxruntime_providers_cpu.cmake)
126-
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_TRT_INTERFACE))
126+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
127127
include(onnxruntime_providers_cuda.cmake)
128128
endif()
129129

130130
if (onnxruntime_USE_DNNL)
131131
include(onnxruntime_providers_dnnl.cmake)
132132
endif()
133133

134-
if (onnxruntime_USE_TENSORRT AND (NOT onnxruntime_ENABLE_TRT_INTERFACE))
134+
if (onnxruntime_USE_TENSORRT AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
135135
include(onnxruntime_providers_tensorrt.cmake)
136136
endif()
137137

138-
if (onnxruntime_USE_VITISAI)
138+
if (onnxruntime_USE_VITISAI AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
139139
include(onnxruntime_providers_vitisai.cmake)
140140
endif()
141141

142-
if (onnxruntime_USE_OPENVINO AND (NOT onnxruntime_ENABLE_OPENVINO_INTERFACE))
142+
if (onnxruntime_USE_OPENVINO AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
143143
include(onnxruntime_providers_openvino.cmake)
144144
endif()
145145

@@ -164,6 +164,7 @@ if (onnxruntime_USE_JSEP)
164164
endif()
165165

166166
if (onnxruntime_USE_QNN)
167+
message("Hitting qnn check")
167168
include(onnxruntime_providers_qnn.cmake)
168169
endif()
169170

cmake/onnxruntime_python.cmake

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -881,7 +881,7 @@ if (onnxruntime_USE_DNNL)
881881
)
882882
endif()
883883

884-
if (onnxruntime_USE_VITISAI)
884+
if (onnxruntime_USE_VITISAI AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
885885
add_custom_command(
886886
TARGET onnxruntime_pybind11_state POST_BUILD
887887
COMMAND ${CMAKE_COMMAND} -E copy
@@ -891,7 +891,7 @@ if (onnxruntime_USE_VITISAI)
891891
)
892892
endif()
893893

894-
if (onnxruntime_USE_TENSORRT)
894+
if (onnxruntime_USE_TENSORRT AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
895895
add_custom_command(
896896
TARGET onnxruntime_pybind11_state POST_BUILD
897897
COMMAND ${CMAKE_COMMAND} -E copy
@@ -911,7 +911,7 @@ if (onnxruntime_USE_MIGRAPHX)
911911
)
912912
endif()
913913

914-
if (onnxruntime_USE_OPENVINO)
914+
if (onnxruntime_USE_OPENVINO AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
915915
add_custom_command(
916916
TARGET onnxruntime_pybind11_state POST_BUILD
917917
COMMAND ${CMAKE_COMMAND} -E copy
@@ -934,7 +934,7 @@ if (DEFINED ENV{OPENVINO_MANYLINUX})
934934
)
935935
endif()
936936

937-
if (onnxruntime_USE_CUDA)
937+
if (onnxruntime_USE_CUDA AND (NOT onnxruntime_ENABLE_GENERIC_INTERFACE))
938938
add_custom_command(
939939
TARGET onnxruntime_pybind11_state POST_BUILD
940940
COMMAND ${CMAKE_COMMAND} -E copy

0 commit comments

Comments
 (0)