Skip to content

Commit c728e10

Browse files
authored
Merge pull request #29225 from compnerd/combine-tensorflow-ctensorflow
TensorFlow: remove the unnecessary code in CTensorFlow
2 parents 627ac35 + 1136b73 commit c728e10

File tree

9 files changed

+37
-356
lines changed

9 files changed

+37
-356
lines changed

stdlib/public/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,6 @@ if(SWIFT_BUILD_STDLIB)
7777
# dependency for `numpy.ndarray` bridging to `ShapedArray`/`Tensor`.
7878
if(SWIFT_ENABLE_TENSORFLOW)
7979
# TODO: Add TensorFlow support for iOS/Raspberry Pi.
80-
add_subdirectory(CTensorFlow)
8180
add_subdirectory(TensorFlow)
8281
endif()
8382
endif()

stdlib/public/CTensorFlow/CMakeLists.txt

Lines changed: 0 additions & 106 deletions
This file was deleted.

stdlib/public/CTensorFlow/ctensorflow_init.cpp

Lines changed: 0 additions & 170 deletions
This file was deleted.

stdlib/public/CTensorFlow/ctensorflow_init.h

Lines changed: 0 additions & 48 deletions
This file was deleted.

stdlib/public/CTensorFlow/module.modulemap

Lines changed: 0 additions & 8 deletions
This file was deleted.

stdlib/public/TensorFlow/CMakeLists.txt

Lines changed: 24 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,10 @@ if (TENSORFLOW_SWIFT_APIS)
4141
endforeach()
4242
endif()
4343

44+
get_filename_component(TensorFlow_LIBRARY_DIR ${TF_LIBRARY} DIRECTORY)
4445
add_swift_target_library(swiftTensorFlow ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES} IS_STDLIB
4546
"${SOURCES}"
4647

47-
INCORPORATE_OBJECT_LIBRARIES swiftCTensorFlow
4848
PRIVATE_LINK_LIBRARIES "${TF_LIBRARIES}"
4949
SWIFT_MODULE_DEPENDS SwiftOnoneSupport
5050
SWIFT_MODULE_DEPENDS_IOS Darwin
@@ -57,10 +57,31 @@ add_swift_target_library(swiftTensorFlow ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES} IS_
5757
SWIFT_MODULE_DEPENDS_HAIKU Glibc
5858
SWIFT_MODULE_DEPENDS_WINDOWS MSVCRT
5959
SWIFT_MODULE_DEPENDS Python
60-
SWIFT_COMPILE_FLAGS "${swift_stdlib_compile_flags}"
61-
LINK_FLAGS "${SWIFT_RUNTIME_SWIFT_LINK_FLAGS}"
60+
SWIFT_COMPILE_FLAGS
61+
${swift_stdlib_compile_flags}
62+
-Xcc -I${CMAKE_CURRENT_SOURCE_DIR}
63+
-Xcc -I${TF_INCLUDE_DIR}
64+
LINK_FLAGS
65+
${SWIFT_RUNTIME_SWIFT_LINK_FLAGS}
66+
-L${TensorFlow_LIBRARY_DIR}
6267
INSTALL_IN_COMPONENT stdlib
6368
EXTRA_RPATHS "${SWIFT_TENSORFLOW_TARGET_LIB_DIR}"
6469
# NOTE: This is a workaround for https://github.com/apple/swift/pull/24382,
6570
# which changed the default install_name_dir to `/usr/bin/swift`.
6671
DARWIN_INSTALL_NAME_DIR "@rpath")
72+
73+
74+
swift_install_in_component(DIRECTORY ${TF_INCLUDE_DIR}/tensorflow/c/
75+
DESTINATION lib/swift/tensorflow/tensorflow/c
76+
COMPONENT stdlib
77+
FILES_MATCHING
78+
PATTERN c_api.h
79+
PATTERN c_api_experimental.h
80+
PATTERN tf_attrtype.h
81+
PATTERN tf_datatype.h
82+
PATTERN tf_status.h
83+
PATTERN tf_tensor.h
84+
PATTERN eager/c_api.h)
85+
swift_install_in_component(FILES module.modulemap CTensorFlow.h
86+
DESTINATION lib/swift/tensorflow
87+
COMPONENT stdlib)
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
#ifndef CTensorFlow_CTensorFlow_h
2+
#define CTensorFlow_CTensorFlow_h
3+
4+
#include <tensorflow/c/c_api.h>
5+
#include <tensorflow/c/c_api_experimental.h>
6+
#include <tensorflow/c/eager/c_api.h>
7+
8+
#endif

0 commit comments

Comments
 (0)