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

Remove unused&deprecated llvm_asm feature #43

Merged
merged 1 commit into from
Jun 9, 2022

Conversation

chinoto
Copy link
Contributor

@chinoto chinoto commented Jun 9, 2022

#38 is closed, but blink is still trying to enable the llvm_asm feature and fails to build. With this, it builds on my machine.

@chinoto chinoto changed the title Remove unused&deprecated feature Remove unused&deprecated llvm_asm feature Jun 9, 2022
@chinoto
Copy link
Contributor Author

chinoto commented Jun 9, 2022

Specified the feature in the PR/commit message >_>

@stappersg
Copy link
Member

mmmm, LOL, it read as I haven't git push ...

@stappersg
Copy link
Member

Yeah, it is better a my

--- a/src/main.rs
+++ b/src/main.rs
@@ -1,4 +1,4 @@
-#![feature(llvm_asm)]
+#![feature(asm_experimental_arch)]
 
 #![no_std]
 #![no_main]

(I'm about to merge)

@stappersg stappersg merged commit 872e48d into avr-rust:master Jun 9, 2022
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

Successfully merging this pull request may close these issues.

2 participants