-
Notifications
You must be signed in to change notification settings - Fork 53
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
[Mac build] Add macros build #845
base: main
Are you sure you want to change the base?
Conversation
5a2d842
to
6ea346c
Compare
747d816
to
992ea9b
Compare
6ea346c
to
fb11672
Compare
425fe14
to
4643a5a
Compare
4643a5a
to
fe272e4
Compare
718402d
to
fb2523c
Compare
992ea9b
to
fc58a43
Compare
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.
I think that you can likely extract the naming changes as that is a purely mechanical change.
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=AMD64" | ||
"extra_flags": "-D CMAKE_SYSTEM_PROCESSOR=AMD64", | ||
"host_built_swiftc": "${HostToolchain}/usr/bin/swiftc.exe", | ||
"macros_swiftflags": "-strict-implicit-module-context -resource-dir ${SDKRoot}/usr/lib/swift -L${SDKRoot}/usr/lib/swift/windows -vfsoverlay ${WINDOWS_VFS_OVERLAY} -Xcc -Xclang -Xcc -fbuiltin-headers-in-system-modules" |
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.
We should separately experiment with this and see if the -strict-implicit-module-context
is needed at all.
22f85d3
to
71128d5
Compare
71128d5
to
116e75d
Compare
4b3edb0
to
eb90037
Compare
0852548
to
6e26d5a
Compare
6e26d5a
to
519271b
Compare
519271b
to
ff39455
Compare
No description provided.