-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Compiler panic on big-endian PowerPC 64 #41443
Comments
triage: P-high @nagisa will do some investigation and try to reproduce. |
I can reproduce this one. Will attempt to reduce. |
A proper backtrace
|
Regression range: FIRST BAD: nightly-2017-03-24 (e703b33 2017-03-23) COMMITS: 8c4f2c6...e703b33 Commit range: EDIT: nightly-2017-03-31 (5c94997 2017-03-30) BAD |
Not specific to PPC64. I'm seeing this also on 32-bit big-endian MIPS (also in qemu) using revision
|
cc @michaelwoerister you sure your |
|
@michaelwoerister The endianness conversion is done ahead of time, to avoid having the byte <-> words conversion functions do any work. EDIT: What's likely the problem here is a missing |
I compiled this on PPC to check and it seems to work, but not sure whether I didn't mess up anything in a major way. Maybe a good shot at rust-lang#41443
Try fixing bigendian metadata serialisation I compiled this on PPC to check and it seems to work, but not sure whether I didn't mess up anything in a major way. Maybe a good shot at #41443 The easiest way to *really* test this is to land the patch (a high priority would be good, since I quite literally am just twiddling my thumbs on this now), wait for nightly, and, if the nightly works, backport this to beta.
The most recent nightly seems to have fixed this. |
Verified fixed in rustc 1.19.0-nightly (6a5fc9e 2017-05-02) on both PPC64 and mips (32-bit big-endian). Thank you. |
I compiled this on PPC to check and it seems to work, but not sure whether I didn't mess up anything in a major way. Maybe a good shot at rust-lang#41443
I compiled this on PPC to check and it seems to work, but not sure whether I didn't mess up anything in a major way. Maybe a good shot at rust-lang#41443
rustc 1.18.0-nightly (ddc5d7b 2017-04-20) on Linux ppc64 (big-endian) panics when compiling a doc test from encoding_rs. No panic on x86_64.
Steps to reproduce:
Use Linux on big-endian PPC 64 and install nightly Rust via rustup. (I used Fedora 25 in qemu; see Linking with panic machinery fails on Linux PPC64 #37996 for how. I used
RUSTUP_USE_HYPER=1
to work around rustup having a broken libcurl at the moment.)Clone encoding_rs and checkout the latest commit at the time of filing this issue:
85f3d89ef81d77f35261f510b56aab023b0b9c46
.Run
RUST_BACKTRACE=1 cargo test
in theencoding_rs
directory.Expected results:
No compiler panic.
Actual results:
The text was updated successfully, but these errors were encountered: