Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Remove 15402 #626

Merged
merged 1 commit into from
Jan 27, 2021
Merged

Remove 15402 #626

merged 1 commit into from
Jan 27, 2021

Conversation

fanninpm
Copy link
Contributor

I noticed this didn't cause an ICE when I was testing out this repository on my new Apple Silicon machine. Seeing as the linked issue is now closed, should we sweep this one out of the way?

The legacy `llvm_asm!` macro is deprecated and no longer maintained.

Issue: rust-lang/rust#15402
Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed this didn't cause an ICE when I was testing out this repository on my new Apple Silicon machine.

I think it's platform-dependent (due to llvm_asm!).

Yeah, I think we could move to fixed as that feature is now unmaintained and wont fix anyway.

@JohnTitor JohnTitor merged commit 7872b31 into rust-lang:master Jan 27, 2021
@fanninpm fanninpm deleted the remove-15402 branch January 28, 2021 01:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants