Skip to content
This repository was archived by the owner on Apr 2, 2020. It is now read-only.

Commit 3a3b6fa

Browse files
author
Harlan Haskins
authored
[swift] Adopt new ModuleInterfaceLoader APIs (#1988)
This patch adopts the new APIs in swiftlang/swift#27175 and removes the usage of `-emit-parseable-module-interface[-path]` in favor of the final flags, `-emit-module-interface[-path]`.
1 parent 9bae308 commit 3a3b6fa

File tree

6 files changed

+21
-21
lines changed

6 files changed

+21
-21
lines changed

include/lldb/lldb-private-enumerations.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -215,11 +215,11 @@ typedef enum LanguageRuntimeDescriptionDisplayVerbosity {
215215
typedef enum SwiftModuleLoadingMode {
216216
eSwiftModuleLoadingModePreferSerialized, // Prefer loading via .swiftmodule,
217217
// falling back to .swiftinterface
218-
eSwiftModuleLoadingModePreferParseable, // Prefer Loading via
218+
eSwiftModuleLoadingModePreferInterface, // Prefer Loading via
219219
// .swiftinterface, falling back to
220220
// .swiftmodule
221221
eSwiftModuleLoadingModeOnlySerialized, // Load via .swiftmodule only
222-
eSwiftModuleLoadingModeOnlyParseable, // Load via .swiftinterface only
222+
eSwiftModuleLoadingModeOnlyInterface, // Load via .swiftinterface only
223223
} SwiftModuleLoadingMode;
224224

225225
//----------------------------------------------------------------------

lit/SwiftREPL/SwiftInterface.test

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
// RUN: rm -rf %t
77
// RUN: mkdir %t
88
// RUN: cp %S/Inputs/A.swift %t/AA.swift
9-
// RUN: %target-swiftc -module-name AA -emit-parseable-module-interface-path %t/AA.swiftinterface -emit-library -o %t/libAA%target-shared-library-suffix %t/AA.swift
9+
// RUN: %target-swiftc -module-name AA -emit-module-interface-path %t/AA.swiftinterface -emit-library -o %t/libAA%target-shared-library-suffix %t/AA.swift
1010
// RUN: rm %t/AA.swift
1111
// RUN: %lldb --repl="-I%t -L%t -lAA" < %s | FileCheck %s
1212

lit/SwiftREPL/SwiftInterfaceForceModuleLoadMode.test

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// This test checks that the value of the symbols.swift-module-loading-mode
22
// setting is respected when loading Swift modules.
3-
// Note: It intentionally does not check the only-parseable or prefer-parseable
3+
// Note: It intentionally does not check the only-interface or prefer-interface
44
// modes as this also causes the Swift stdlib to be loaded via its module
55
// interface file, which slows down this test considerably.
66

@@ -18,7 +18,7 @@
1818
// RUN: mkdir %t/mcp
1919
// RUN: mkdir %t/lib
2020
// RUN: cp %S/Inputs/A.swift %t/AA.swift
21-
// RUN: %target-swiftc -module-name AA -emit-parseable-module-interface-path %t/lib/AA.swiftinterface -emit-library -o %t/lib/libAA%target-shared-library-suffix %t/AA.swift
21+
// RUN: %target-swiftc -module-name AA -emit-module-interface-path %t/lib/AA.swiftinterface -emit-library -o %t/lib/libAA%target-shared-library-suffix %t/AA.swift
2222
// RUN: sed -e 's/FromInterface/FromSerialized/g' %t/AA.swift | %target-swiftc -module-name AA -emit-module -o %t/lib/AA.swiftmodule -
2323
// RUN: rm %t/AA.swift
2424

packages/Python/lldbsuite/test/make/Makefile.rules

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -694,7 +694,7 @@ $(MODULENAME).swiftmodule: $(OBJECTS) $(DYLIB_OBJECTS)
694694
@echo "### Merging swift modules for $(MODULENAME)"
695695
$(SWIFT_FE) $(SWIFT_FEFLAGS) -merge-modules \
696696
-emit-module $(patsubst %.swift,%.partial.swiftmodule,$(SWIFT_SOURCES) $(DYLIB_SWIFT_SOURCES)) \
697-
-emit-parseable-module-interface-path $(BUILDDIR)/$(MODULENAME).swiftinterface \
697+
-emit-module-interface-path $(BUILDDIR)/$(MODULENAME).swiftinterface \
698698
-sil-merge-partial-modules \
699699
-disable-diagnostic-passes -disable-sil-perf-optzns \
700700
-module-name $(MODULENAME) \

source/Core/ModuleList.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,13 @@ static bool KeepLookingInDylinker(SymbolContextList &sc_list, size_t start_idx);
6969
namespace {
7070

7171
static constexpr OptionEnumValueElement g_swift_module_loading_mode_enums[] = {
72-
{eSwiftModuleLoadingModePreferParseable, "prefer-parseable",
72+
{eSwiftModuleLoadingModePreferInterface, "prefer-interface",
7373
"Prefer loading Swift modules via their .swiftinterface file, but fall back "
7474
" to the .swiftmodule if it is missing."},
7575
{eSwiftModuleLoadingModePreferSerialized, "prefer-serialized",
7676
"Prefer loading Swift module via their .swiftmodule file if present, but "
7777
"fall back to the .swiftinterface if it is missing or invalid (default)."},
78-
{eSwiftModuleLoadingModeOnlyParseable, "only-parseable",
78+
{eSwiftModuleLoadingModeOnlyInterface, "only-interface",
7979
"Only load Swift modules via their .swiftinterface file - ignore "
8080
".swiftmodule files."},
8181
{eSwiftModuleLoadingModeOnlySerialized, "only-serialized",

source/Symbol/SwiftASTContext.cpp

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
#include "swift/Demangling/Demangle.h"
4646
#include "swift/Demangling/ManglingMacros.h"
4747
#include "swift/Frontend/Frontend.h"
48-
#include "swift/Frontend/ParseableInterfaceModuleLoader.h"
48+
#include "swift/Frontend/ModuleInterfaceLoader.h"
4949
#include "swift/Frontend/PrintingDiagnosticConsumer.h"
5050
#include "swift/IRGen/Linking.h"
5151
#include "swift/SIL/SILModule.h"
@@ -3442,14 +3442,14 @@ swift::ASTContext *SwiftASTContext::GetASTContext() {
34423442
case eSwiftModuleLoadingModePreferSerialized:
34433443
loading_mode = swift::ModuleLoadingMode::PreferSerialized;
34443444
break;
3445-
case eSwiftModuleLoadingModePreferParseable:
3446-
loading_mode = swift::ModuleLoadingMode::PreferParseable;
3445+
case eSwiftModuleLoadingModePreferInterface:
3446+
loading_mode = swift::ModuleLoadingMode::PreferInterface;
34473447
break;
34483448
case eSwiftModuleLoadingModeOnlySerialized:
34493449
loading_mode = swift::ModuleLoadingMode::OnlySerialized;
34503450
break;
3451-
case eSwiftModuleLoadingModeOnlyParseable:
3452-
loading_mode = swift::ModuleLoadingMode::OnlyParseable;
3451+
case eSwiftModuleLoadingModeOnlyInterface:
3452+
loading_mode = swift::ModuleLoadingMode::OnlyInterface;
34533453
break;
34543454
}
34553455

@@ -3465,21 +3465,21 @@ swift::ASTContext *SwiftASTContext::GetASTContext() {
34653465
m_ast_context_ap->addModuleLoader(std::move(memory_buffer_loader_ap));
34663466
}
34673467

3468-
// 2. Create and install the parseable interface module loader.
3468+
// 2. Create and install the module interface loader.
34693469
//
3470-
// TODO: It may be nice to reverse the order between PIML and SML in
3470+
// TODO: It may be nice to reverse the order between MIL and SML in
34713471
// LLDB, since binary swift modules likely contain private
3472-
// types that the parseable interfaces are missing. On the
3472+
// types that the module interfaces are missing. On the
34733473
// other hand if we need to go looking for a module on disk,
34743474
// something is already screwed up in the debug info.
3475-
std::unique_ptr<swift::ModuleLoader> parseable_module_loader_ap;
3475+
std::unique_ptr<swift::ModuleLoader> module_interface_loader_ap;
34763476
if (loading_mode != swift::ModuleLoadingMode::OnlySerialized) {
3477-
std::unique_ptr<swift::ModuleLoader> parseable_module_loader_ap(
3478-
swift::ParseableInterfaceModuleLoader::create(
3477+
std::unique_ptr<swift::ModuleLoader> module_interface_loader_ap(
3478+
swift::ModuleInterfaceLoader::create(
34793479
*m_ast_context_ap, moduleCachePath, prebuiltModuleCachePath,
34803480
tracker, loading_mode));
3481-
if (parseable_module_loader_ap)
3482-
m_ast_context_ap->addModuleLoader(std::move(parseable_module_loader_ap));
3481+
if (module_interface_loader_ap)
3482+
m_ast_context_ap->addModuleLoader(std::move(module_interface_loader_ap));
34833483
}
34843484

34853485
// 3. Create and install the serialized module loader.

0 commit comments

Comments
 (0)