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

Add support for our temporary target #68

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Conversation

iamrecursion
Copy link
Collaborator

Summary

For the current phase of work we are requiring that input LLVM IR be built to target the aarch64-unknown-none-softfloat target. This means that it can make no assumptions about either the host operating system (as there is none), nor the target ABI (meaning that we fall back to platform codegen ABI convention, and hence that rustc will not do anything funky here).

Details

We add support in the nix derivation (flake.nix) so that this target is automatically made available in the sandbox. It may be necessary to add it at the system level as well, depending on how your development environment is set up.

Checklist

  • Code is formatted by Rustfmt.
  • Documentation has been updated if necessary.

@iamrecursion iamrecursion added the enhancement New feature or request label Sep 30, 2024
@iamrecursion iamrecursion self-assigned this Sep 30, 2024
@iamrecursion iamrecursion requested a review from a team as a code owner September 30, 2024 20:38
@iamrecursion iamrecursion force-pushed the wip/ara/target-platform branch from e7970c2 to 60fc8ea Compare September 30, 2024 20:44
For the current phase of work we are requiring that input LLVM IR be
built to target the `aarch64-unknown-none-softfloat` target. This means
that it can make no assumptions about either the host operating system
(as there is none), nor the target ABI (meaning that we fall back to
platform codegen ABI convention, and hence that `rustc` will not do
anything funky here).

We add support in the nix derivation (`flake.nix`) so that this target
is automatically made available in the sandbox. It may be necessary to
add it at the system level as well, depending on how your development
environment is set up.
@iamrecursion iamrecursion force-pushed the wip/ara/target-platform branch from 60fc8ea to 0e15ee9 Compare September 30, 2024 20:45
Copy link
Contributor

@wzmuda wzmuda left a comment

Choose a reason for hiding this comment

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

lgtm

@iamrecursion iamrecursion merged commit 12eb081 into main Sep 30, 2024
4 checks passed
@iamrecursion iamrecursion deleted the wip/ara/target-platform branch September 30, 2024 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants