Skip to content

[AllocStackHoisting] New dealloc_stack's locs are cleanups. #40516

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

nate-chandler
Copy link
Contributor

AllocStackHoisting creates new dealloc_stack instructions. The old dealloc_stack instructions' locations' kinds are ::CleanupKind The new instructions' locations need to be of that same kind too in order not to trigger the sil verifier failure

Basic block contains a non-contiguous lexical scope at -Onone

when building at Onone.

AllocStackHoisting creates new dealloc_stack instructions.  The old
dealloc_stack instructions' locations' kinds are ::CleanupKind  The new
instructions' locations need to be of that same kind too in order not to
trigger the sil verifier failure

    Basic block contains a non-contiguous lexical scope at -Onone

when building at Onone.
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler nate-chandler requested a review from atrick December 11, 2021 00:29
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.

Thanks!

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 78babbc

@nate-chandler
Copy link
Contributor Author

@swift-ci please clean test linux platform

@nate-chandler nate-chandler merged commit 103cc32 into swiftlang:main Dec 11, 2021
@nate-chandler nate-chandler deleted the alloc_stack_hoisting/make-new-dealloc_stacks-locs-cleanups branch December 11, 2021 21:03
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.

3 participants