Skip to content

[LLVM 6.0] LowerFormalArguments didn't emit the correct number of values #102

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

Closed
jhwgh1968 opened this issue May 3, 2018 · 4 comments
Closed
Labels
A-libcore Affects compiling the core library A-llvm Affects the LLVM AVR backend

Comments

@jhwgh1968
Copy link

Reduced test case:

; ModuleID = 'bugpoint-reduced-simplified.bc'
source_filename = "bugpoint-output-7b39239.bc"
target datalayout = "e-p:16:8-i8:8-i16:8-i32:8-i64:8-f32:8-f64:8-n8-a:8"
target triple = "avr-unknown-unknown"

; Function Attrs: nounwind readnone uwtable
define { i64, i16 } @"_ZN59_$LT$f32$u20$as$u20$core..num..dec2flt..rawfp..RawFloat$GT$6unpack17h2977bc660656d2faE"(float %self) unnamed_addr #1 {
start:
  ret { i64, i16 } zeroinitializer
}

The full error message:

llc: /home/admin/Software/avr-rust/src/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:8736: void llvm::SelectionDAGISel::LowerArguments(const llvm::Function&): Assertion `InVals.size() == Ins.size() && "LowerFormalArguments didn't emit the correct number of values!"' failed.

This is after applying #92 to @shepmaster's rebased branch.

cc @brainlag

@brainlag
Copy link

brainlag commented May 3, 2018

#worksforme

You are missing a patch, probably this one

Duplicate of #92.

@shepmaster
Copy link
Member

Yeah, that seemingly never got into LLVM, either... I'll see about pushing up some new versions with more of the recent patches applied.

@jhwgh1968
Copy link
Author

That is the only patch I did apply, actually. It got me past the original problem of running out of registers.

Having said that, I would not be surprised if I am missing something else. I guess I'll wait for @shepmaster to update his branch with all the known fixes across the issue tracker before I start digging for more bugs.

@jhwgh1968
Copy link
Author

I figured out there was a second patch for #92 that I did not apply, as suspected. Closing.

@shepmaster shepmaster added A-libcore Affects compiling the core library A-llvm Affects the LLVM AVR backend and removed A-libcore Affects compiling the core library labels May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-libcore Affects compiling the core library A-llvm Affects the LLVM AVR backend
Projects
None yet
Development

No branches or pull requests

3 participants