Skip to content

follow up #20109; remove shallow seqs/strings for ORC #20502

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
merged 5 commits into from
Oct 6, 2022
Merged

Conversation

ringabout
Copy link
Member

follow up #20109

@ringabout ringabout changed the title remove shallow seqs/strings for ORC follow up #20109; remove shallow seqs/strings for ORC Oct 5, 2022
@ringabout
Copy link
Member Author

@Varriount Varriount requested a review from Araq October 5, 2022 17:59
@Varriount Varriount added the Requires Araq To Merge PR should only be merged by Araq label Oct 5, 2022
@Araq Araq merged commit 723a71b into devel Oct 6, 2022
@Araq Araq deleted the pr_shallow branch October 6, 2022 05:16
@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

Thanks for your hard work on this PR!
The lines below are statistics of the Nim compiler built from 723a71b

Hint: mm: orc; opt: speed; options: -d:release
162678 lines; 9.335s; 665.09MiB peakmem

capocasa pushed a commit to capocasa/Nim that referenced this pull request Mar 31, 2023
…lang#20502)

* remove `shallow` seqs/strings for ORC

* add a changelog item

* change url of DelaunayNim
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Araq To Merge PR should only be merged by Araq
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants