Skip to content

Commit 5aa63fc

Browse files
hyphyp
authored
Merge pull request #72634 from hyp/eng/android/ndk-overlay
[android] add an Android NDK Swift overlay
2 parents 46502f9 + 743a501 commit 5aa63fc

File tree

72 files changed

+305
-31
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

72 files changed

+305
-31
lines changed

‎lib/ClangImporter/ClangIncludePaths.cpp‎

Lines changed: 20 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ createClangArgs(const ASTContext &ctx, clang::driver::Driver &clangDriver) {
180180

181181
static SmallVector<std::pair<std::string, std::string>, 2>
182182
getLibcFileMapping(ASTContext &ctx, StringRef modulemapFileName,
183-
std::optional<StringRef> maybeHeaderFileName,
183+
std::optional<ArrayRef<StringRef>> maybeHeaderFileNames,
184184
const llvm::IntrusiveRefCntPtr<llvm::vfs::FileSystem> &vfs) {
185185
const llvm::Triple &triple = ctx.LangOpts.Target;
186186

@@ -220,18 +220,20 @@ getLibcFileMapping(ASTContext &ctx, StringRef modulemapFileName,
220220
SmallVector<std::pair<std::string, std::string>, 2> vfsMappings{
221221
{std::string(injectedModuleMapPath), std::string(actualModuleMapPath)}};
222222

223-
if (maybeHeaderFileName) {
224-
// TODO: remove the SwiftGlibc.h header and reference all Glibc headers
225-
// directly from the modulemap.
226-
Path actualHeaderPath = actualModuleMapPath;
227-
llvm::sys::path::remove_filename(actualHeaderPath);
228-
llvm::sys::path::append(actualHeaderPath, maybeHeaderFileName.value());
223+
if (maybeHeaderFileNames) {
224+
for (const auto &filename : *maybeHeaderFileNames) {
225+
// TODO: remove the SwiftGlibc.h header and reference all Glibc headers
226+
// directly from the modulemap.
227+
Path actualHeaderPath = actualModuleMapPath;
228+
llvm::sys::path::remove_filename(actualHeaderPath);
229+
llvm::sys::path::append(actualHeaderPath, filename);
229230

230-
Path injectedHeaderPath(libcDir);
231-
llvm::sys::path::append(injectedHeaderPath, maybeHeaderFileName.value());
231+
Path injectedHeaderPath(libcDir);
232+
llvm::sys::path::append(injectedHeaderPath, filename);
232233

233-
vfsMappings.push_back(
234-
{std::string(injectedHeaderPath), std::string(actualHeaderPath)});
234+
vfsMappings.push_back(
235+
{std::string(injectedHeaderPath), std::string(actualHeaderPath)});
236+
}
235237
}
236238

237239
return vfsMappings;
@@ -552,9 +554,14 @@ ClangInvocationFileMapping swift::getClangInvocationFileMapping(
552554
} else if (triple.isMusl()) {
553555
libcFileMapping =
554556
getLibcFileMapping(ctx, "musl.modulemap", StringRef("SwiftMusl.h"), vfs);
557+
} else if (triple.isAndroid()) {
558+
// Android uses the android-specific module map that overlays the NDK.
559+
StringRef headerFiles[] = {"SwiftAndroidNDK.h", "SwiftBionic.h"};
560+
libcFileMapping =
561+
getLibcFileMapping(ctx, "android.modulemap", headerFiles, vfs);
555562
} else if (triple.isOSGlibc() || triple.isOSOpenBSD() ||
556-
triple.isOSFreeBSD() || triple.isAndroid()) {
557-
// Android/BSD/Linux Mappings
563+
triple.isOSFreeBSD()) {
564+
// BSD/Linux Mappings
558565
libcFileMapping = getLibcFileMapping(ctx, "glibc.modulemap",
559566
StringRef("SwiftGlibc.h"), vfs);
560567

‎stdlib/cmake/modules/AddSwiftStdlib.cmake‎

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1757,6 +1757,9 @@ endfunction()
17571757
# SWIFT_MODULE_DEPENDS_WASI
17581758
# Swift modules this library depends on when built for WASI.
17591759
#
1760+
# SWIFT_MODULE_DEPENDS_ANDROID
1761+
# Swift modules this library depends on when built for Android.
1762+
#
17601763
# FRAMEWORK_DEPENDS
17611764
# System frameworks this library depends on.
17621765
#
@@ -1919,6 +1922,7 @@ function(add_swift_target_library name)
19191922
SWIFT_COMPILE_FLAGS_XROS
19201923
SWIFT_COMPILE_FLAGS_LINUX
19211924
SWIFT_MODULE_DEPENDS
1925+
SWIFT_MODULE_DEPENDS_ANDROID
19221926
SWIFT_MODULE_DEPENDS_CYGWIN
19231927
SWIFT_MODULE_DEPENDS_FREEBSD
19241928
SWIFT_MODULE_DEPENDS_FREESTANDING
@@ -2146,12 +2150,15 @@ function(add_swift_target_library name)
21462150
elseif(sdk STREQUAL "OPENBSD")
21472151
list(APPEND swiftlib_module_depends_flattened
21482152
${SWIFTLIB_SWIFT_MODULE_DEPENDS_OPENBSD})
2149-
elseif(sdk STREQUAL "LINUX" OR sdk STREQUAL "ANDROID")
2153+
elseif(sdk STREQUAL "LINUX")
21502154
list(APPEND swiftlib_module_depends_flattened
21512155
${SWIFTLIB_SWIFT_MODULE_DEPENDS_LINUX})
21522156
elseif(sdk STREQUAL "LINUX_STATIC")
21532157
list(APPEND swiftlib_module_depends_flattened
21542158
${SWIFTLIB_SWIFT_MODULE_DEPENDS_LINUX_STATIC})
2159+
elseif(sdk STREQUAL "ANDROID")
2160+
list(APPEND swiftlib_module_depends_flattened
2161+
${SWIFTLIB_SWIFT_MODULE_DEPENDS_ANDROID})
21552162
elseif(sdk STREQUAL "CYGWIN")
21562163
list(APPEND swiftlib_module_depends_flattened
21572164
${SWIFTLIB_SWIFT_MODULE_DEPENDS_CYGWIN})
@@ -3005,6 +3012,7 @@ function(add_swift_target_executable name)
30053012
DEPENDS
30063013
LINK_LIBRARIES
30073014
SWIFT_MODULE_DEPENDS
3015+
SWIFT_MODULE_DEPENDS_ANDROID
30083016
SWIFT_MODULE_DEPENDS_CYGWIN
30093017
SWIFT_MODULE_DEPENDS_FREEBSD
30103018
SWIFT_MODULE_DEPENDS_FREESTANDING
@@ -3118,12 +3126,15 @@ function(add_swift_target_executable name)
31183126
elseif(sdk STREQUAL "OPENBSD")
31193127
list(APPEND swiftexe_module_depends_flattened
31203128
${SWIFTEXE_TARGET_SWIFT_MODULE_DEPENDS_OPENBSD})
3121-
elseif(sdk STREQUAL "LINUX" OR sdk STREQUAL "ANDROID")
3129+
elseif(sdk STREQUAL "LINUX")
31223130
list(APPEND swiftexe_module_depends_flattened
31233131
${SWIFTEXE_TARGET_SWIFT_MODULE_DEPENDS_LINUX})
31243132
elseif(sdk STREQUAL "LINUX_STATIC")
31253133
list(APPEND swiftexe_module_depends_flattened
31263134
${SWIFTEXE_TARGET_SWIFT_MODULE_DEPENDS_LINUX_STATIC})
3135+
elseif(sdk STREQUAL "ANDROID")
3136+
list(APPEND swiftexe_module_depends_flattened
3137+
${SWIFTEXE_TARGET_SWIFT_MODULE_DEPENDS_ANDROID})
31273138
elseif(sdk STREQUAL "CYGWIN")
31283139
list(APPEND swiftexe_module_depends_flattened
31293140
${SWIFTEXE_TARGET_SWIFT_MODULE_DEPENDS_CYGWIN})

‎stdlib/private/RuntimeUnittest/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ add_swift_target_library(swiftRuntimeUnittest ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES
88
ExclusivityTests.cpp
99

1010
SWIFT_MODULE_DEPENDS StdlibUnittest
11+
SWIFT_MODULE_DEPENDS_ANDROID Android
1112
SWIFT_MODULE_DEPENDS_LINUX Glibc
1213
SWIFT_MODULE_DEPENDS_FREEBSD Glibc
1314
SWIFT_MODULE_DEPENDS_OPENBSD Glibc

‎stdlib/private/StdlibCollectionUnittest/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ add_swift_target_library(swiftStdlibCollectionUnittest ${SWIFT_STDLIB_LIBRARY_BU
1919
WriteBackMutableSlice.swift
2020

2121
SWIFT_MODULE_DEPENDS StdlibUnittest
22+
SWIFT_MODULE_DEPENDS_ANDROID Android
2223
SWIFT_MODULE_DEPENDS_LINUX Glibc
2324
SWIFT_MODULE_DEPENDS_FREEBSD Glibc
2425
SWIFT_MODULE_DEPENDS_OPENBSD Glibc

‎stdlib/private/StdlibUnicodeUnittest/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ add_swift_target_library(swiftStdlibUnicodeUnittest ${SWIFT_STDLIB_LIBRARY_BUILD
1010
WordBreaking.swift
1111

1212
SWIFT_MODULE_DEPENDS StdlibUnittest
13+
SWIFT_MODULE_DEPENDS_ANDROID Android
1314
SWIFT_MODULE_DEPENDS_LINUX Glibc
1415
SWIFT_MODULE_DEPENDS_FREEBSD Glibc
1516
SWIFT_MODULE_DEPENDS_OPENBSD Glibc

‎stdlib/private/StdlibUnittest/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ add_swift_target_library(swiftStdlibUnittest ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES}
6262
SWIFT_MODULE_DEPENDS_XROS ${swift_stdlib_unittest_darwin_dependencies}
6363
SWIFT_MODULE_DEPENDS_MACCATALYST ${swift_stdlib_unittest_darwin_dependencies}
6464
SWIFT_MODULE_DEPENDS_FREESTANDING "${SWIFT_FREESTANDING_TEST_DEPENDENCIES}"
65+
SWIFT_MODULE_DEPENDS_ANDROID Android
6566
SWIFT_MODULE_DEPENDS_LINUX Glibc
6667
SWIFT_MODULE_DEPENDS_FREEBSD Glibc
6768
SWIFT_MODULE_DEPENDS_OPENBSD Glibc

‎stdlib/private/StdlibUnittest/RaceTest.swift‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ import Darwin
4545
import Glibc
4646
#elseif canImport(Musl)
4747
import Musl
48+
#elseif canImport(Android)
49+
import Android
4850
#elseif os(WASI)
4951
import WASILibc
5052
#elseif os(Windows)

‎stdlib/private/StdlibUnittest/StdlibCoreExtras.swift‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ import Darwin
1818
import Glibc
1919
#elseif canImport(Musl)
2020
import Musl
21+
#elseif canImport(Android)
22+
import Android
2123
#elseif os(WASI)
2224
import WASILibc
2325
#elseif os(Windows)

‎stdlib/private/StdlibUnittest/StdlibUnittest.swift‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ import Darwin
2424
import Glibc
2525
#elseif canImport(Musl)
2626
import Musl
27+
#elseif canImport(Android)
28+
import Android
2729
#elseif os(WASI)
2830
import WASILibc
2931
#elseif os(Windows)

‎stdlib/private/SwiftPrivate/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ add_swift_target_library(swiftSwiftPrivate ${SWIFT_STDLIB_LIBRARY_BUILD_TYPES} I
2727
SWIFT_MODULE_DEPENDS_WATCHOS ${swift_swiftprivate_darwin_depencencies}
2828
SWIFT_MODULE_DEPENDS_MACCATALYST ${swift_swiftprivate_darwin_depencencies}
2929
SWIFT_MODULE_DEPENDS_FREESTANDING "${SWIFT_FREESTANDING_TEST_DEPENDENCIES}"
30+
SWIFT_MODULE_DEPENDS_ANDROID Android
3031
SWIFT_MODULE_DEPENDS_LINUX Glibc
3132
SWIFT_MODULE_DEPENDS_LINUX_STATIC Musl
3233
SWIFT_MODULE_DEPENDS_FREEBSD Glibc

0 commit comments

Comments
 (0)