Skip to content

Commit 23b82ce

Browse files
committed
Re-run makeOptions to sync up Swift and C++ driver
There's been a few PRs mentioning that running `makeOptions` results in more churn than the one option they were expecting. Run separately to sync up again.
1 parent a3364a1 commit 23b82ce

File tree

3 files changed

+52
-29
lines changed

3 files changed

+52
-29
lines changed

Sources/SwiftDriver/Jobs/FrontendJobHelpers.swift

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,7 @@ extension Driver {
116116
}
117117

118118
// Add flags for C++ interop
119-
if parsedOptions.hasArgument(.enableExperimentalCxxInterop) {
120-
commandLine.appendFlag(.enableCxxInterop)
121-
}
119+
try commandLine.appendLast(.enableExperimentalCxxInterop, from: &parsedOptions)
122120
if let stdlibVariant = parsedOptions.getLastArgument(.experimentalCxxStdlib)?.asSingle {
123121
commandLine.appendFlag("-Xcc")
124122
commandLine.appendFlag("-stdlib=\(stdlibVariant)")
@@ -152,7 +150,6 @@ extension Driver {
152150
try commandLine.appendLast(.enableLibraryEvolution, from: &parsedOptions)
153151
try commandLine.appendLast(.enableTesting, from: &parsedOptions)
154152
try commandLine.appendLast(.enablePrivateImports, from: &parsedOptions)
155-
try commandLine.appendLast(.enableCxxInterop, from: &parsedOptions)
156153
try commandLine.appendLast(in: .g, from: &parsedOptions)
157154
try commandLine.appendLast(.debugInfoFormat, from: &parsedOptions)
158155
try commandLine.appendLast(.importUnderlyingModule, from: &parsedOptions)

0 commit comments

Comments
 (0)