Skip to content

[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

Merged
merged 1 commit into from
Jun 30, 2021

Conversation

mshockwave
Copy link
Contributor

@mshockwave mshockwave commented Jun 29, 2021

The -gsil is deprecated in favor of -sil-based-debuginfo.

This PR is a synergy with swiftlang/swift#38167.

@mshockwave
Copy link
Contributor Author

@swift-ci test

@nkcsgexi nkcsgexi self-requested a review June 29, 2021 22:52
Copy link
Contributor

@adrian-prantl adrian-prantl left a 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.

@adrian-prantl
Copy link
Contributor

We may have an exhaustive list of the existing -gsil userbase in the list of Reviewers now 😀

Copy link
Contributor

@atrick atrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet!

@adrian-prantl
Copy link
Contributor

@mshockwave For the odd chance that there are other users of -gsil: Since we don't use -gsil for anything else — should we define it and print a "this option is deprecated" error or alias it to the new option?

@mshockwave
Copy link
Contributor Author

@mshockwave For the odd chance that there are other users of -gsil: Since we don't use -gsil for anything else — should we define it and print a "this option is deprecated" error or alias it to the new option?

Yes I can do that

@mshockwave
Copy link
Contributor Author

@mshockwave For the odd chance that there are other users of -gsil: Since we don't use -gsil for anything else — should we define it and print a "this option is deprecated" error or alias it to the new option?

Yes I can do that

Please refer to swiftlang/swift#38167 (comment)

@swift-ci test

Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nkcsgexi
Copy link
Contributor

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`.
@mshockwave mshockwave changed the title [SIL] Simplify debug info generation flow for SIL files [SIL] Deprecating the '-gsil' frontend flag Jun 30, 2021
@mshockwave
Copy link
Contributor Author

Reflecting the updates in swiftlang/swift#38167 , the driver is no longer passing -Xllvm -sil-print-debuginfo if -g is given.

@swift-ci test

@mshockwave mshockwave merged commit eeda545 into swiftlang:main Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants