Comment on thisIsGoingToBeASeriousDebate
Speiser0@feddit.de 11 months ago
Definitely left. Right one won’t be optimized. (And there are so many mistakes in your inline asm…)
Comment on thisIsGoingToBeASeriousDebate
Speiser0@feddit.de 11 months ago
Definitely left. Right one won’t be optimized. (And there are so many mistakes in your inline asm…)
TadoTheRustacean@programming.dev 11 months ago
What mistakes?
Speiser0@feddit.de 11 months ago
Mostly the missing listing of clobbered registers. Other than that it’s mostly just that you’re doing useless things, like manually putting the stuff into the registers instead of letting the compiler do it, and the useless push and pop. And the loop is obviously not needed and would hurt performance if you do every write like that.
(“so many” was inappropriate, sorry.)
TadoTheRustacean@programming.dev 11 months ago
I am hopeless at getting the text_ptr simpler than i64::from_str_radix(&format!(“{:p}”, my_string)[2…], 16).unwrap(); How can i get it the normal way?
Speiser0@feddit.de 11 months ago
Just use
str::as_ptr()
.Here’s an example (disclaimer: I haven’t used inline asm in rust before, expect issues): godbolt.org/z/sczYGe96f