-
Notifications
You must be signed in to change notification settings - Fork 10.5k
runtime: use std::max_align_t
over max_align_t
(NFC)
#17068
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
MSVC does not provide a full C99 environment. Use the C++ equivalent spelling for the value. This allows the build to succeed on Windows.
@swift-ci please test and merge |
@compnerd Looks like this broke Ubuntu 14.04 bot - https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-14_04/4486/ |
Weird, |
@compnerd I think you need to |
Can we revert, or correct this with a new PR that is tested on 14.04? |
Maybe #17101 is going to fix it? |
@millenomi - trying to setup a 14.04 environment to look into this. We should certainly look into correcting this :-) The interesting thing is that the change does include |
Looks like my changes didn't help but it's good to have them anyway... |
There's a stddef.h in include/swift/Runtime/Portability.h too. Is that (transitively) included? |
Maybe, I've opened #17126 to deal with that as well... |
I'm still trying to build in a VM :-( |
I didn't expect building against 14.04 to be so challenging. Unfortunately, this change is needed to build for Windows, so both states are less than ideal. The solution seems to be that we need to special case the 14.04 build. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56019 indicates that GCC 4.8 has a bug in the C++11 standard library implementation and is missing the definition of |
I'm happy to hold off on merging the revert if you can land something relatively quickly that resolves the issue, but we shouldn't let this keep the CI red for much longer. |
MSVC does not provide a full C99 environment. Use the C++ equivalent
spelling for the value. This allows the build to succeed on Windows.
Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.
Resolves SR-NNNN.