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.

Sign up for GitHub

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

Jump to bottom

Inline assembly should switch to llvm_asm! #204

Closed
jonas-schievink opened this issue Apr 12, 2020 · 0 comments · Fixed by #210
Closed

Inline assembly should switch to llvm_asm! #204

jonas-schievink opened this issue Apr 12, 2020 · 0 comments · Fixed by #210
Labels
help wanted

Comments

@jonas-schievink
Copy link
Contributor

asm! will be deprecated soon rust-lang/rust#71007
@adamgreig adamgreig mentioned this issue Apr 21, 2020
Merged
@stianeklund stianeklund mentioned this issue Apr 28, 2020
Merged
bors bot added a commit that referenced this issue May 5, 2020
@bors @stianeklund
Merge #210
1e29ce5 
210: Use llmv_asm! macro for inline assembly r=jonas-schievink a=stianeklund

Fixes #204

the `asm!` macro will soon be deprecated.
This changes all of the `asm!` calls to `llvm_asm!`

To my knowledge doing a direct replacement should be fine.


Co-authored-by: Stian Eklund <[email protected]>
@bors bors bot closed this as completed in ba7496c May 5, 2020
@little-arhat little-arhat mentioned this issue Jun 7, 2020
Closed
jonas-schievink pushed a commit that referenced this issue Jul 19, 2020
@stianeklund @jonas-schievink
Use llmv_asm! macro for inline assembly
4add5a5 
\ fixes #204

the `asm!` macro will soon be deprecated.
This changes all of the `asm!` calls to `llvm_asm!`
To my knowledge doing a direct replacement should be fine.
@TDHolmes TDHolmes mentioned this issue Dec 15, 2021
Open
adamgreig pushed a commit that referenced this issue Jan 12, 2022
@bors @jonas-schievink
Merge #204
9214dda 
204: Slightly improve the bash scripts r=adamgreig a=jonas-schievink

cf. #165

Co-authored-by: Jonas Schievink <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use llmv_asm! macro for inline assembly stianeklund/cortex-m
1 participant
@jonas-schievink