Skip to content

[pull] swiftwasm-release/5.3 from release/5.3 #1157

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 4 commits into from
Jun 5, 2020

Conversation

pull[bot]
Copy link

@pull pull bot commented Jun 4, 2020

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

Nathan Hawes and others added 3 commits June 3, 2020 13:30
…nt a crash during indexing.

locator->getAnchor() may return a null Expr*, but was being passed directly to
isa<KeyPathExpr>(..) without a null check in two places.

Resovles rdar://problem/63558609
…lang#31968)

* [5.3] IRGen: Always eliminate frame pointers of leaf functions

Explanation: This change makes it such that we don't emit frame pointers
in leaf functions leading to decreased code size and improved
performance.

Scope: Should not visibly affect Swift programs other than making them
faster.

Risk: Medium. There might be unknown dependencies on having the frame
pointer available in leaf functions.

Testing: Unit test added

Reviewer: Andrew Trick

rdar://20933449

* Fix test

* Only use omit the frame pointer on arm64, x86_64 darwin needs it for some its tools

* Fix test on linux
…p-indexing-crash

[5.3] Fix dynamic member lookup index-while-building crash
@MaxDesiatov MaxDesiatov merged commit 9a06725 into swiftwasm-release/5.3 Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants