Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Cannot bootstrap for PowerPC with LLVM assertions enabled #36150

Closed
alexcrichton opened this issue Aug 30, 2016 · 2 comments · Fixed by #62949
Closed

Cannot bootstrap for PowerPC with LLVM assertions enabled #36150

alexcrichton opened this issue Aug 30, 2016 · 2 comments · Fixed by #62949
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. C-bug Category: This is a bug. O-PowerPC Target: PowerPC processors

Comments

@alexcrichton
Copy link
Member

alexcrichton commented Aug 30, 2016 •
edited

Relevant code to disable this

From these logs.

rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/x86_64-unknown-linux-gnu/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/aarch64-unknown-linux-gnu/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/arm-unknown-linux-gnueabi/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/arm-unknown-linux-gnueabihf/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/armv7-unknown-linux-gnueabihf/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/mips-unknown-linux-gnu/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/mips-unknown-linux-musl/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/mipsel-unknown-linux-gnu/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/mipsel-unknown-linux-musl/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/powerpc-unknown-linux-gnu/lib/libcore
rustc: x86_64-unknown-linux-gnu/stage2/lib/rustlib/powerpc64-unknown-linux-gnu/lib/libcore
Invalid PPC CTR loop!
UNREACHABLE executed at /buildslave/rust-buildbot/slave/auto-linux-cross-opt/build/src/llvm/lib/Target/PowerPC/PPCCTRLoops.cpp:722!
Aborted (core dumped)
make: *** [x86_64-unknown-linux-gnu/stage2/lib/rustlib/powerpc-unknown-linux-gnu/lib/stamp.core] Error 134

This means we do not blanket turn on LLVM assertions on all of our bots, which we would ideally like to do. I'm going to try turning them off for just the cross bot though.
@alexcrichton
Copy link
Member Author

cc rust-lang-deprecated/rust-buildbot@571864e

cc @eddyb

@alexcrichton alexcrichton added the A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. label Aug 30, 2016
@alexcrichton alexcrichton mentioned this issue Aug 30, 2016
Closed
@nagisa
Copy link
Member

nagisa commented Aug 31, 2016

Seems related to https://llvm.org/bugs/show_bug.cgi?id=27083 somewhat. We do not use the maxnum intrinsic (totally should!) but from the existence of bug for maxnum seems to suggest that a similar issue for some non-maxnum intrinsic is likely to exist (and libcore is kinda intrinsic-heavy).

@sanxiyn sanxiyn added the O-PowerPC Target: PowerPC processors label Jan 10, 2017
@Mark-Simulacrum Mark-Simulacrum added the C-bug Category: This is a bug. label Jul 26, 2017
@mati865 mati865 mentioned this issue Jul 24, 2019
Merged
Centril added a commit to Centril/rust that referenced this issue Jul 26, 2019
@Centril
Rollup merge of rust-lang#62949 - mati865:patch-1, r=alexcrichton
394f049 
Re-enable assertions in PPC dist builder

Fixes rust-lang#36150

Log of successful build:
https://dev.azure.com/mati865/6518b167-4cf6-4587-b3d1-8e137f2fb2e4/_apis/build/builds/23/logs/825
Centril added a commit to Centril/rust that referenced this issue Jul 26, 2019
@Centril
Rollup merge of rust-lang#62949 - mati865:patch-1, r=alexcrichton
2fa3e20 
Re-enable assertions in PPC dist builder

Fixes rust-lang#36150

Log of successful build:
https://dev.azure.com/mati865/6518b167-4cf6-4587-b3d1-8e137f2fb2e4/_apis/build/builds/23/logs/825
Centril added a commit to Centril/rust that referenced this issue Jul 27, 2019
@Centril
Rollup merge of rust-lang#62949 - mati865:patch-1, r=alexcrichton
7f74000 
Re-enable assertions in PPC dist builder

Fixes rust-lang#36150

Log of successful build:
https://dev.azure.com/mati865/6518b167-4cf6-4587-b3d1-8e137f2fb2e4/_apis/build/builds/23/logs/825
Centril added a commit to Centril/rust that referenced this issue Jul 27, 2019
@Centril
Rollup merge of rust-lang#62949 - mati865:patch-1, r=alexcrichton
a686323 
Re-enable assertions in PPC dist builder

Fixes rust-lang#36150

Log of successful build:
https://dev.azure.com/mati865/6518b167-4cf6-4587-b3d1-8e137f2fb2e4/_apis/build/builds/23/logs/825
Centril added a commit to Centril/rust that referenced this issue Jul 28, 2019
@Centril
Rollup merge of rust-lang#62949 - mati865:patch-1, r=alexcrichton
fb6fdd4 
Re-enable assertions in PPC dist builder

Fixes rust-lang#36150

Log of successful build:
https://dev.azure.com/mati865/6518b167-4cf6-4587-b3d1-8e137f2fb2e4/_apis/build/builds/23/logs/825
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. C-bug Category: This is a bug. O-PowerPC Target: PowerPC processors
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Re-enable assertions in PPC dist builder mati865/rust
4 participants
@sanxiyn @alexcrichton @nagisa @Mark-Simulacrum