-
Notifications
You must be signed in to change notification settings - Fork 204
[SIL] Deprecating the '-gsil' frontend flag #737
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 |
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.
LGTM form my side, but please wait for @nkcsgexi to also approve the PR.
We may have an exhaustive list of the existing |
95d3903
to
09bb2cf
Compare
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.
Sweet!
@mshockwave For the odd chance that there are other users of |
Yes I can do that |
09bb2cf
to
595f1e7
Compare
Please refer to swiftlang/swift#38167 (comment) @swift-ci test |
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.
LGTM!
The patch looks great to me! For future reference, we don't have to update the option list if the driver code doesn't use any new options in the implementation. The option list will be eventually updated by later changes. |
The `-gsil` is deprecated in favor of `-sil-based-debuginfo`.
595f1e7
to
1d9dc53
Compare
Reflecting the updates in swiftlang/swift#38167 , the driver is no longer passing @swift-ci test |
The
-gsil
is deprecated in favor of-sil-based-debuginfo
.This PR is a synergy with swiftlang/swift#38167.