Skip to content
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

Backport safepoints at function entry #83

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

kpamnany
Copy link
Collaborator

@kpamnany kpamnany commented Sep 27, 2023

PR Description

Backport JuliaLang#41616 and JuliaLang#47393 as an attempt to fix https://relationalai.atlassian.net/browse/RAI-13268.

Checklist

Requirements for merging:

@github-actions github-actions bot added port-to-v1.10 This change should apply to Julia v1.10 builds port-to-master This change should apply to all future Julia builds port-to-v1.9 This change should apply to Julia v1.9 builds labels Sep 27, 2023
@kpamnany kpamnany removed the port-to-master This change should apply to all future Julia builds label Sep 27, 2023
@kpamnany kpamnany marked this pull request as ready for review September 28, 2023 14:27
@kpamnany kpamnany force-pushed the kp-backport-41616-47393 branch 2 times, most recently from f7dfc94 to dd676d6 Compare September 28, 2023 19:18
@kpamnany kpamnany requested a review from a team September 28, 2023 23:56
* Emit safepoints at function entry

* Make safepoint emission on function entry a codegen feature

* Hoist signal page lookup outside fence

* Update src/cgutils.cpp

* Fix rebase
@kpamnany kpamnany merged commit 2452a59 into v1.9.2+RAI Oct 6, 2023
1 check passed
@kpamnany kpamnany deleted the kp-backport-41616-47393 branch October 6, 2023 21:07
@kpamnany kpamnany removed port-to-v1.9 This change should apply to Julia v1.9 builds port-to-v1.10 This change should apply to Julia v1.10 builds labels Oct 19, 2023
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.

3 participants