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
CI: use DEVELOPMENT-SNAPSHOT-2019-01-23-a #780
CI: use DEVELOPMENT-SNAPSHOT-2019-01-23-a #780
Conversation
Woop CI failure. |
Huh, negative amount of remaining allocations, will investigate 🤣 |
d8abe26
to
86bd77b
Compare
yaay, @Lukasa the inliner threshold must have changed, but now it finally passes 🙌 |
@@ -74,6 +74,9 @@ private func writeTrailers(wrapOutboundOut: (IOData) -> NIOAny, ctx: ChannelHand | |||
} | |||
} | |||
|
|||
// starting about swift-5.0-DEVELOPMENT-SNAPSHOT-2019-01-20-a, this doesn't get automatically inlined, which costs | |||
// 2 extra allocations so we need to help the optimiser out. | |||
@inline(__always) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eeckstein between swift-5.0-DEVELOPMENT-SNAPSHOT-2019-01-13-a and swift-5.0-DEVELOPMENT-SNAPSHOT-2019-01-20-a this suddenly didn't get inlined anymore without any code changes.
Is that expected? It was stable from at least Swift 4.0.1 until now and then suddenly last week it broke...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and thanks again @aschwaighofer for apple/swift#21933 which will make debugging that sort of thing unnecessary in the future :)
86bd77b
to
83479ca
Compare
Motivation:
Newer Swift versions are good and we should test them, this one threw a few curveballs at us:
posix_memalign
which needed fixing in our alloc tests (fixed in alloc tests: hook posix_memalign & reallocf #782)Modifications
@inline(__always)
to help the optimiserResult: