Skip to content

Commit cf7b7c7

Browse files
authored
Merge pull request #66370 from xymus/fix-serial-macros-5.9
[5.9][Serialization] Fix serializing CompilerPluginExecutablePaths values
2 parents 96d78b5 + 8123bff commit cf7b7c7

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

lib/Frontend/Frontend.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ SerializationOptions CompilerInvocation::computeSerializationOptions(
225225
llvm::interleave(
226226
pair.ModuleNames, [&](auto &name) { optStr += name; },
227227
[&]() { optStr += ","; });
228-
serializationOpts.CompilerPluginLibraryPaths.push_back(optStr);
228+
serializationOpts.CompilerPluginExecutablePaths.push_back(optStr);
229229
}
230230

231231
serializationOpts.DisableCrossModuleIncrementalInfo =

lib/Serialization/Serialization.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1150,7 +1150,7 @@ void Serializer::writeHeader(const SerializationOptions &options) {
11501150

11511151
options_block::CompilerPluginExecutablePathLayout
11521152
CompilerPluginExecutablePath(Out);
1153-
for (auto Arg : options.CompilerPluginLibraryPaths) {
1153+
for (auto Arg : options.CompilerPluginExecutablePaths) {
11541154
CompilerPluginExecutablePath.emit(ScratchRecord, Arg);
11551155
}
11561156
}

test/Macros/serialize_plugin_search_paths.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
// CHECK: -plugin-path: {{.*}}plugins
1616
// CHECK: -plugin-path: {{.*}}plugins
1717
// CHECK: -plugin-path: {{.*}}plugins
18+
// CHECK: -plugin-path: {{.*}}plugins
1819
// CHECK: -external-plugin-path: {{.*}}plugins#{{.*}}swift-plugin-server
1920
// CHECK: -load-plugin-library: {{.*}}MacroDefinition.{{dylib|so|dll}}
20-
// CHECK: -load-plugin-executable: {{.*}}MacroDefinition.{{dylib|so|dll}}
2121
// CHECK: -load-plugin-executable: {{.*}}mock-plugin#TestPlugin

0 commit comments

Comments
 (0)