-
Notifications
You must be signed in to change notification settings - Fork 287
Remove the workarounds for some resolved issues #1774
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
base: master
Are you sure you want to change the base?
Conversation
There seems to be some instruction check failures (specifically |
I'd just remove the tests on those (set the assert_instr to |
f4c53d7
to
5d9e007
Compare
There are a few more changes in ASM, specifically
@Amanieu are these acceptable? |
That looks like a regression on the LLVM side, it should be emitting the For now this can be accepted with a FIXME, but an issue should be opened in LLVM about it. |
5b4c8f4
to
675eca1
Compare
Seems like rust-lang/rust#60637 is still causing some problems in LLVM optimizing for darwin and ios, so I am delegating that |
8c237c1
to
62ff5cc
Compare
dox
CI forarmv7-unknown-linux-gnueabihf
vfmaddcsh
intrinsics llvm/llvm-project#98306