-
Notifications
You must be signed in to change notification settings - Fork 13k
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 building Redox #32783
Comments
We are getting this in Redox travis too: https://travis-ci.org/redox-os/redox/builds/121316042 It looks like it affects OS X and FreeBSD as well as Linux https://github.com/redox-os/redox/issues/606#issuecomment-206649111 |
Older version of rustc doesn't have this error:
|
Got problems here as well.
|
make all in redox succeeds
make all in redox fails |
Same here.
Current redox commit is |
FWIW, also seeing the above reported issue
|
What is the simplest way for me to reproduce this problem? Does anyone have a narrowed test case of any kind? The instructions at https://github.com/redox-os/redox seem complex (run some script, run qemu etc) -- it'd be great if there were some rust file I could build without all those steps. Any tips? |
I'm sure, you can just clone Redox repo (git clone git@github.com:redox-os/redox.git --origin upstream --recursive; git submodule update --recursive --init), install nasm and try to build project (make all). |
Based on @yohanesu75's comment, I examined the commits between the two nightlies, and the most suspicious was #32742. I've built a version of rustc based on e8ab71f (right before the merge), compiled Redox succesfully, built another version based on 953c3b5 (merge commit of #32742), with which I was able to reproduce the ICE. cc @eddyb |
Very interesting @pmarcelll. I am working to find the code in libstd that triggers it |
I found it, it is alloc_system.rs Trying to find a fix, the file can be seen here: https://github.com/redox-os/redox/blob/master/libstd/src/alloc_system.rs |
@pmarcelll I see what the problem is. Writing the test is going to be the tricky part, I believe I need two |
trans: always register an item's symbol, even if duplicated. Fixes rust-lang#32783 which was introduced by not always registering item symbols in rust-lang#32742.
I tried to build current Redox OS HEAD with last available rustc-nightly and got an ICE.
Meta
rustc --version --verbose
:Backtrace:
The text was updated successfully, but these errors were encountered: