Skip to content

/rust

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

LLVM invalid linkage error from nested closures #24779

Closed
solson opened this Issue Apr 24, 2015 · 0 comments

Comments

Assignees
No one assigned
Labels
A-LLVM T-compiler
Projects
None yet
Milestone
No milestone
3 participants
@solson @steveklabnik @nikomatsakis
@solson
Copy link
Member

solson commented Apr 24, 2015

Code:

(||||42)()()

Error:

Global is external, but doesn't have external or weak linkage!
i32 (%closure*)* @_ZN4main12closure.1067E
invalid linkage type for function declaration
i32 (%closure*)* @_ZN4main12closure.1067E
LLVM ERROR: Broken module found, compilation aborted!

@steveklabnik steveklabnik added I-wrong A-LLVM labels Apr 28, 2015

@nikomatsakis nikomatsakis added the T-compiler label Jun 3, 2015

eefriedman added a commit to eefriedman/rust that referenced this issue Jun 7, 2015

@eefriedman
Translate "ignored" closure expressions. 
This isn't a very clean fix, but I'm not sure what a better fix would look
like.

Fixes rust-lang#24779.
428a396

@eefriedman eefriedman referenced this issue Jun 7, 2015

Merged

Translate "ignored" closure expressions. #26079

eefriedman added a commit to eefriedman/rust that referenced this issue Jun 8, 2015

@eefriedman
Translate "ignored" closure expressions. 
This isn't a very clean fix, but I'm not sure what a better fix would look
like.

Fixes rust-lang#24779.
2442f83

bors added a commit that referenced this issue Jun 8, 2015

@bors
Auto merge of #26079 - eefriedman:emit-closure, r=nrc 
This isn't a very clean fix, but I'm not sure what a better fix would look
like.

Fixes #24779.
Loading status checks…
521f82e

@bors bors closed this in #26079 Jun 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.