-
Notifications
You must be signed in to change notification settings - Fork 204
Make the DWARF version configurable. #1467
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@swift-ci test with swiftlang/swift#69254 |
test with swiftlang/swift#69254 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One suggested cleanup to remove the unsightly !
s, but otherwise this LGTM
@@ -2308,7 +2308,24 @@ extension Driver { | |||
diagnosticsEngine.emit(.error_argument_not_allowed_with(arg: fullNotAllowedOption, other: levelOption.spelling)) | |||
} | |||
|
|||
return DebugInfo(format: format, level: level, shouldVerify: shouldVerify) | |||
// Determine the DWARF version. | |||
var dwarfVersion: UInt8 = 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usually we just use Int
, but this is okay.
5c568d8
to
4900dda
Compare
test with swiftlang/swift#69254 |
test with swiftlang/swift#69254 |
This mirrors a similar change in the swift repository.
4900dda
to
ac0ab85
Compare
test with swiftlang/swift#69254 |
test with swiftlang/swift#69254 |
@swift-ci test windows |
This mirrors a similar change in the swift repository.