Skip to content

Switch to using llvm_asm! instead of asm! #351

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

Merged
merged 3 commits into from
Apr 29, 2020

Conversation

alexcrichton
Copy link
Member

No description provided.

@alexcrichton alexcrichton merged commit cde22bc into rust-lang:master Apr 29, 2020
@alexcrichton alexcrichton deleted the switch-to-llvm-asm branch April 29, 2020 20:30
@Amanieu
Copy link
Member

Amanieu commented May 11, 2020

Could you release a new version of compiler-builtins with this change? It is blocking rust-lang/rust#69171.

@alexcrichton
Copy link
Member Author

Sure thing, done now!

@alexcrichton
Copy link
Member Author

As an FYI as well libs is added to this crate on crates.io (and core too) in case publishes are needed and I'm not around. There's some instructions to publish in the repo as well. I certainly don't mind continuing to publish, but if I'm unresponsive or busy I'd want to make sure I'm not accidentally blocking anything!

@Amanieu
Copy link
Member

Amanieu commented May 12, 2020

I tried to do it myself but I am missing direct push access to the repo, which is needed to push a new tag for the version.

@alexcrichton
Copy link
Member Author

Oh good point, should be fixed now!

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