Open itamarsch opened 2 months ago
I think:
main.rs
PLUS Cargo.toml
and such files for reproducablityFirst of all, you really should be avoiding the use of f64
on AVR microcontrollers. This will eat up tons and tons of memory because all floating point operations need to be implemented in software. If you really need floats, f32 is at least not quite as bloaty...
That said, the error still shouldn't happen. It is most likely a compiler issue of some sorts, though. Ping @Patryk27 maybe you know something?
It looks like I've accidentally forgotten to #[avr_skip]
this intrinsic way back in 2023 👀
(tl;dr this function gets provided both by GCC's standard library and Rust's compiler-builtins, which creates a linking conflict)
I've created https://github.com/rust-lang/compiler-builtins/pull/601 to fix this; after it gets merged, I'll bump compiler-builtins within rustc and ping back, as always 🙂
It looks like I've accidentally forgotten to
#[avr_skip]
this intrinsic way back in 2023 👀(tl;dr this function gets provided both by GCC's standard library and Rust's compiler-builtins, which creates a linking conflict)
I've created https://github.com/rust-lang/compiler-builtins/pull/601 to fix this; after it gets merged, I'll bump compiler-builtins within rustc and ping back, as always 🙂
In rustc where is the version of compiler-builtins defined? I'm not really familiar with the rustc codebase
Right here :-)
It looks like I've accidentally forgotten to
#[avr_skip]
this intrinsic way back in 2023 👀(tl;dr this function gets provided both by GCC's standard library and Rust's compiler-builtins, which creates a linking conflict)
I've created https://github.com/rust-lang/compiler-builtins/pull/601 to fix this; after it gets merged, I'll bump compiler-builtins within rustc and ping back, as always 🙂
@Patryk27 Have you bumped up compiler-builtins?
I'm on it, but there are some (AVR-unrelated) crashes that I need to investigate.
Status: together with my AVR fix, bumping compiler-builtins brought some extra (non-AVR-related) f16 & f128 changes which need special handling in the compiler, so it's not as easy of a merge as usual - the current pull request is: