Skip to content

Commit aaa7270

Browse files
committed
Adjust 'makeOptions' to new TableGen format of Swift's 'Options.inc'
1 parent 39151f6 commit aaa7270

File tree

4 files changed

+106
-48
lines changed

4 files changed

+106
-48
lines changed

Package.swift

+6-2
Original file line numberDiff line numberDiff line change
@@ -129,9 +129,13 @@ let package = Package(
129129
/// The `makeOptions` utility (for importing option definitions).
130130
.executableTarget(
131131
name: "makeOptions",
132-
dependencies: []),
132+
dependencies: [],
133+
// Do not enforce checks for LLVM's ABI-breaking build settings.
134+
// makeOptions runtime uses some header-only code from LLVM's ADT classes,
135+
// but we do not want to link libSupport into the executable.
136+
cxxSettings: [.unsafeFlags(["-DLLVM_DISABLE_ABI_BREAKING_CHECKS_ENFORCING=1"])]),
133137
],
134-
cxxLanguageStandard: .cxx14
138+
cxxLanguageStandard: .cxx17
135139
)
136140

137141
if ProcessInfo.processInfo.environment["SWIFT_DRIVER_LLBUILD_FWK"] == nil {

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ $ apt-get install libncurses-dev
156156
be found, e.g.:
157157

158158
```
159-
$ swift build -Xcc -I/path/to/build/Ninja-ReleaseAssert/swift-.../include --product makeOptions
159+
$ swift build -Xcc -Xcc -I/path/to/build/Ninja-Release/swift-.../include -Xcc -I/path/to/build/Ninja-Release/llvm-.../include -Xcc -I/path/to/source/llvm-project/llvm/include --product makeOptions
160160
```
161161

162162
Then, run `makeOptions` and redirect the output to overwrite `Options.swift`:

0 commit comments

Comments
 (0)