-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Closed
Labels
P-highHigh priorityHigh priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.regression-from-stable-to-nightlyPerformance or correctness regression from stable to nightly.Performance or correctness regression from stable to nightly.
Description
As reported here:
https://users.rust-lang.org/t/mandel-rust-v0-4/7481/3
Compiling this little project:
https://github.com/willi-kappler/mandel-rust
On x86_64-pc-windows-gnu, rustc 1.17.0-nightly, 691eba1 2017-03-01:
Instruction does not dominate all uses!
%27 = load i32, i32* %26, !range !2
%71 = icmp eq i32 %27, 60
LLVM ERROR: Broken function found, compilation aborted!
error: Could not compile clap
.
Further notes:
I can confirm this on MacOS and Linux. Stable and beta ran fine, but nightly has problems on both platforms
Metadata
Metadata
Assignees
Labels
P-highHigh priorityHigh priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.regression-from-stable-to-nightlyPerformance or correctness regression from stable to nightly.Performance or correctness regression from stable to nightly.
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
TimNN commentedon Mar 9, 2017
Only happens with
--release
for me on macOS.nagisa commentedon Mar 9, 2017
First bad:
1572bf104 2017-02-25
Last good:
08230775a 2017-02-24
Changes
TimNN commentedon Mar 9, 2017
Note that while the error does not occur when everything is compiled in debug mode, it does occur when clap itself is compiled with
-Copt-level=0
but all the dependencies are-Copt-level=3
.nagisa commentedon Mar 9, 2017
It also sounds a lot like an LLVM bug, since the issue happens during LLVM passes – should check if LLVM 4.0 works or not. Us giving LLVM more information is most likely the trigger.My bad! It is us who are generating bad IR!
TimNN commentedon Mar 9, 2017
@nagisa: This also fails with LLVM 4.0
nagisa commentedon Mar 9, 2017
Minimal-ish reproducer:
Another interesting case is
Which ICEs:
This points pretty squarely on the normalize breaks PR.
nagisa commentedon Mar 9, 2017
cc @cramertj
nikomatsakis commentedon Mar 9, 2017
cc me
cramertj commentedon Mar 9, 2017
Is this a dup of #40235? (Fixed in #40242)
nagisa commentedon Mar 10, 2017
nagisa commentedon Mar 10, 2017
Reproduced, rather. I recommend trying to re-reproduce.
cramertj commentedon Mar 10, 2017
@nagisa Doing that now. The easy way to check if you have the bugfix from #40242 is to see if this tries to compile (it did before the bugfix, but not after):
nagisa commentedon Mar 11, 2017
Seems to be fixed in master, so all we need is a nightly release (our current is from 03-03)
cramertj commentedon Mar 11, 2017
That was what I found, too. Sorry, forgot to post my results.
steveklabnik commentedon Mar 14, 2017
#40214 (comment) does not reproduce on the current nightly. closing!