-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[ClangImporter] Fix handling of bitfields in unions #14412
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
[ClangImporter] Fix handling of bitfields in unions #14412
Conversation
@swift-ci Please test |
Build failed |
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.
Looks good, thanks Jordan!
@swift-ci Please test Linux |
Build failed |
(here) Okay, this doesn't seem possible. The Clang importer unconditionally loads that module map. @swift-ci Please test Linux |
Build failed |
At least it reproduces reliably. Looking into it. |
rdar://problem/37242238
ed372b6
to
e435e93
Compare
Filed LLVM PR36245. Full Mac tests passed, so @swift-ci Please test Linux |
@swift-ci Please smoke test macOS |
Build failed |
Previously this caused an assertion failure in +Asserts builds and incorrect behavior in -Asserts builds. rdar://problem/37242238 (cherry picked from commit c06cc01)
Previously this caused an assertion failure in +Asserts builds and incorrect behavior in -Asserts builds. rdar://problem/37242238 (cherry picked from commit c06cc01)
Previously this caused an assertion failure in +Asserts builds and incorrect behavior in -Asserts builds.
rdar://problem/37242238