-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Attempt to generate a next toolchain with #61668 #61721
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
Attempt to generate a next toolchain with #61668 #61721
Conversation
and match the usage pattern employed by other LLVM projects. For context about the underlying change see https://reviews.llvm.org/D117977 Addresses rdar://101396797
Also ensure we don't symlink to swift-frontend twice
@swift-ci please build toolchain |
@swift-ci please build toolchain |
Use CMAKE_INSTALL_BINDIR as initial value for SWIFT_TOOLS_INSTALL_DIR
…he function name for exported functions. More recent versions of Clang (~July 2022) have stricter enforcement of the positioning of attributes; for example, the `nonnull` and `nodiscard` attributes can no longer precede the `extern "C"` specifier and must follow the name of the function (and not the signature of the function) to which they apply. (cherry picked from commit 2963c38)
c738e09
to
923fc30
Compare
@swift-ci please build toolchain |
Hitting the following error on macOS
|
Hitting the following error on Linux
|
Hitting the following on Windows
|
@swift-ci please build toolchain Linux |
In Linux I was able to go past the failing point, now we are failing building Swift -- possibly related to https://reviews.llvm.org/D131717
|
@swift-ci please build toolchain Linux |
That somehow worked, and now I'm getting errors similar to the Windows run
|
Trying to get a sense if my proposed change works in principle
Base PR
add_swift_tool_symlink
to install symlink forswift-frontend
#61668Cherry-picks
Runtime: correct attribute annotations for newer compilers #61489SWIFT_RETURNS_NONNULL
andSWIFT_NODISCARD
attributes after the function name for exported functions. #61473