Skip to content

[lldb] Make sure to eliminate ownership after the diagnostic passes, … #114

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

Conversation

gottesmm
Copy link

@gottesmm gottesmm commented Nov 3, 2019

…but before IRGen.

Before I flip the switch, this will be a no-op. It is needed though after I flip
the switch since IRgen is not meant to see ownership SIL.

…but before IRGen.

Before I flip the switch, this will be a no-op. It is needed though after I flip
the switch since IRgen is not meant to see ownership SIL.
@gottesmm gottesmm requested a review from dcci November 3, 2019 21:15
@gottesmm
Copy link
Author

gottesmm commented Nov 3, 2019

This will be a no-op before I flip the switch for moving ownership lowering after diagnostics. After I flip the switch, it will be necessary since IRGen is not setup to see ownership SIL.

@gottesmm
Copy link
Author

gottesmm commented Nov 3, 2019

@swift-ci test

@gottesmm gottesmm merged commit 0bf3ac5 into swiftlang:swift/master Nov 3, 2019
@gottesmm gottesmm deleted the swift/master-ownership-lowering branch November 3, 2019 23:29
Copy link
Member

@dcci dcci left a comment

Choose a reason for hiding this comment

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

We discussed this offline, but for the records, LGTM.

swift-ci pushed a commit that referenced this pull request May 7, 2023
swift-ci pushed a commit that referenced this pull request May 10, 2023
…-extensions

MSVC makes these string literals [1][2].

[1] https://godbolt.org/z/6vnTzbExx
[2] https://learn.microsoft.com/en-us/cpp/preprocessor/predefined-macros?view=msvc-170

Fixes #114

Initial commit didn't check if there was a function name when stepping through expressions ignoring parens.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D146764
swift-ci pushed a commit that referenced this pull request Nov 8, 2024
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