Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

fix: use arch instead of os for allocator #113

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

doinkythederp
Copy link
Member

Describe the changes this PR makes. Why should it be merged?

This change is a step in updating cargo-pros to use the "none" OS in its target file, which is required to get certain compiler builtins to generate like fmaxf.

Additional Context

  • I have tested these changes on a VEX V5 brain.

Copy link
Member

@Gavin-Niederman Gavin-Niederman left a comment

Choose a reason for hiding this comment

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

LGTM

@Gavin-Niederman Gavin-Niederman merged commit 12b9941 into main Apr 5, 2024
6 checks passed
@Gavin-Niederman Gavin-Niederman deleted the update-target-os branch April 5, 2024 22:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants