Skip to content

Revert "[IRGen] Use the lexical decl context for VarDecls." #33813

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

Closed
wants to merge 1 commit into from

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Sep 5, 2020

Reverts #33306

@compnerd
Copy link
Member Author

compnerd commented Sep 5, 2020

@swift-ci please smoke test

@3405691582
Copy link
Member

Sorry; do I need to create a new PR to include the fix?

@compnerd
Copy link
Member Author

compnerd commented Sep 5, 2020

@3405691582 oh, you already have a fix? in that case, could you please create a PR? we should get that merged instead.

@3405691582
Copy link
Member

Added a PR to fix forward; I can cherry pick in the linked PR and recommit on that PR if that's preferred.

@compnerd compnerd closed this Sep 5, 2020
@compnerd
Copy link
Member Author

compnerd commented Sep 5, 2020

Superseded by #33816

@compnerd compnerd deleted the revert-33306-FixAssertingLocalExtern branch September 5, 2020 01:16
@compnerd compnerd restored the revert-33306-FixAssertingLocalExtern branch September 5, 2020 02:00
@compnerd compnerd reopened this Sep 5, 2020
3405691582 added a commit to 3405691582/swift that referenced this pull request Sep 5, 2020
Windows IR seems to include a `dso_local` annotation, which the check
didn't expect and caused a failure in swiftlang#33813. Added a regex check to
ensure the test passes with or without the annotation.
@compnerd compnerd closed this Sep 5, 2020
@compnerd compnerd deleted the revert-33306-FixAssertingLocalExtern branch September 5, 2020 14:35
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.

2 participants