301

thisIsGoingToBeASeriousDebate (programming.dev)

submitted 1 year ago by TadoTheRustacean@programming.dev to c/programmer_humor@programming.dev

40 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[-] Speiser0@feddit.de 15 points 1 year ago* (last edited 1 year ago)

Definitely left. Right one won't be optimized. (And there are ~~so many~~ some mistakes in your inline asm...)

[-] TadoTheRustacean@programming.dev 1 points 1 year ago

What mistakes?

[-] Speiser0@feddit.de 6 points 1 year 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.

asm!(
"syscall",
in("rax") 1,
in("rdi") 1,
in("rsi") text_ptr,
in("rdx") text_size,

)

("so many" was inappropriate, sorry.)

[-] TadoTheRustacean@programming.dev 1 points 1 year 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?