Skip to content

avoid using core_intrinsics feature (for a transition) - #1500

Closed
RalfJung wants to merge 1 commit into
rust-lang:masterfrom
RalfJung:no-core-intrinsics
Closed

avoid using core_intrinsics feature (for a transition)#1500
RalfJung wants to merge 1 commit into
rust-lang:masterfrom
RalfJung:no-core-intrinsics

Conversation

@RalfJung

Copy link
Copy Markdown
Member

This should let us do the transition in rust-lang/rust#118123.

@rustbot

rustbot commented Nov 21, 2023

Copy link
Copy Markdown
Collaborator

r? @Amanieu

(rustbot has picked a reviewer for you, use r? to override)

@RalfJung

Copy link
Copy Markdown
Member Author

Hm, that test failure looks unrelated to my change?

@lu-zero

lu-zero commented Nov 21, 2023

Copy link
Copy Markdown
Contributor

It can be a qemu glitch.

@Amanieu

Amanieu commented Nov 22, 2023

Copy link
Copy Markdown
Member

I restarted the tests, but I think it might be a real failure. Perhaps something got updated on the LLVM side and it now generates a different instruction?

@RalfJung

Copy link
Copy Markdown
Member Author

Renaming this feature is a big hassle, so I've changed the rustc PR to avoid doing that.

@RalfJung RalfJung closed this Nov 22, 2023
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.

4 participants