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

Mark llvm_asm calls as volatile #13

Closed
Yarn opened this issue Apr 5, 2021 · 1 comment
Closed

Mark llvm_asm calls as volatile #13

Yarn opened this issue Apr 5, 2021 · 1 comment

Comments

@Yarn
Copy link

Yarn commented Apr 5, 2021

I'm pretty sure the llvm_asm invocation should be marked as volatile like

unsafe {llvm_asm!("1: sbiw $0,1
   brne 1b"
   :
   : "w" (0)
   :
   : "volatile")}

This could potentially explain #11 and avr-rust/blink#34 but I don't have an AVR device set up to test with.

@stappersg
Copy link
Member

llvm_asm!() has been replaced with asm!().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants