Skip to content

[IRGen] Add main() to __swift5_entry. #33246

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

Merged

Conversation

nate-chandler
Copy link
Contributor

@nate-chandler nate-chandler commented Aug 1, 2020

To enable lookup of the entry point, emit a pointer to that entry point into a new section; on MachO, the __swift5_entry section of the __TEXT segment.

rdar://problem/66402358

@nate-chandler nate-chandler requested a review from jckarter August 1, 2020 01:33
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch 2 times, most recently from a152626 to 63dcab6 Compare August 3, 2020 16:57
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test OS X Platform
Git Sha - 323843f1daf56d86ced7307ac089a207f6e02f8d

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test Linux Platform
Git Sha - 323843f1daf56d86ced7307ac089a207f6e02f8d

@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch 2 times, most recently from 249bb4a to bd84c07 Compare August 3, 2020 18:11
@nate-chandler nate-chandler changed the title [IRGen] Add main() to __swift_entry. [IRGen] Add main() to __swift5_entry. Aug 3, 2020
@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch from bd84c07 to 78d8b18 Compare August 3, 2020 18:17
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test Linux Platform
Git Sha - 63dcab68e95a69cbfbc3cb5055294a83a83c39bd

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test OS X Platform
Git Sha - 63dcab68e95a69cbfbc3cb5055294a83a83c39bd

@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch from 78d8b18 to a87890a Compare August 3, 2020 18:38
Copy link
Contributor

@aschwaighofer aschwaighofer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@nate-chandler nate-chandler marked this pull request as ready for review August 3, 2020 18:43
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler
Copy link
Contributor Author

@swift-ci please test windows

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test OS X Platform
Git Sha - 78d8b1865285b3f5e9dec4d71a45d97271698551

@swift-ci
Copy link
Contributor

swift-ci commented Aug 3, 2020

Build failed
Swift Test Linux Platform
Git Sha - 78d8b1865285b3f5e9dec4d71a45d97271698551

@nate-chandler nate-chandler requested a review from jckarter August 3, 2020 19:07
@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch from a87890a to 1210f4e Compare August 3, 2020 21:52
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler
Copy link
Contributor Author

@swift-ci please test windows

To enable lookup of the entry point, emit a pointer to that entry point
into a new section; on MachO, the __swift5_entry section of the __TEXT
segment.

rdar://problem/66402358
@nate-chandler nate-chandler force-pushed the emit-main-into-entry-segment branch from 1210f4e to 28cc144 Compare August 3, 2020 22:40
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler
Copy link
Contributor Author

@swift-ci please test windows

@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler
Copy link
Contributor Author

@swift-ci please test windows

@nate-chandler nate-chandler merged commit de4872b into swiftlang:master Aug 4, 2020
@nate-chandler nate-chandler deleted the emit-main-into-entry-segment branch August 4, 2020 02:52
3405691582 added a commit to 3405691582/swift that referenced this pull request Sep 2, 2020
In swiftlang#33246 among other changes, a CHECK referring to the size of
_swift1_autolink_entries was changed from a regex-based expectation to a
static value. This means on platforms with different link flags, this
expectation fails but does not signify any real failure. Revert back to
using a regex check for the length of this constant.
3405691582 added a commit to 3405691582/swift that referenced this pull request Dec 11, 2020
In swiftlang#33246 among other changes, a CHECK referring to the size of
_swift1_autolink_entries was changed from a regex-based expectation to a
static value. This means on platforms with different link flags, this
expectation fails but does not signify any real failure. Revert back to
using a regex check for the length of this constant.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants