-
Notifications
You must be signed in to change notification settings - Fork 314
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
feat: allow for compilation on aarch64 #1204
Conversation
fil-proofs-tooling/src/metadata.rs
Outdated
@@ -109,6 +110,33 @@ impl SystemMetadata { | |||
memory_total_bytes: memory.total().get::<uom::si::information::byte>(), | |||
}) | |||
} | |||
|
|||
#[cfg(not(target_arch = "x86_64"))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally prefer keeping the cfg()
differences as small as possible. We could create a methos that returns the host, memory and the cpu_logical/physical that is shared between the new()
s. This would also make it more apparent what the difference between the cfgs is. Though it''s a matter of test, hence just a comment and nothing that needs to change/blocking a merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this could be nicer, but I mostly wanted to get this to compile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
more fixes sha2raw: update deps and enable asm by default patch ff fix compile time imports missing import move prefetch into macro fixup fil-proofs-tooling refactoring refactoring
@@ -25,7 +25,7 @@ chrono = { version = "0.4.7", features = ["serde"] } | |||
memmap = "0.7.0" | |||
bellperson = "0.9.1" | |||
paired = "0.20.0" | |||
fil-sapling-crypto = "0.6.0" | |||
fil-sapling-crypto = "0.6.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rust oddity, these individual patch version updates can be omitted. They don't hurt though ;-)
Depends on