Skip to content

[LLVM 4.0] Update rustllvm to understand the new DIBulder::createBasicType API#38294

Closed
dylanmckay wants to merge 1 commit into
rust-lang:masterfrom
dylanmckay:llvm-4.0-createbasictype
Closed

[LLVM 4.0] Update rustllvm to understand the new DIBulder::createBasicType API#38294
dylanmckay wants to merge 1 commit into
rust-lang:masterfrom
dylanmckay:llvm-4.0-createbasictype

Conversation

@dylanmckay

Copy link
Copy Markdown
Contributor

This lets Rust work with the createBasicType API in LLVM 4.0.

The alignment argument was dropped in D25073.

This lets Rust work with the createBasicType API in LLVM 4.0.

The alignment argument was dropped in D25073.

- https://reviews.llvm.org/D25073
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@dylanmckay dylanmckay mentioned this pull request Dec 11, 2016
23 tasks
@dylanmckay

Copy link
Copy Markdown
Contributor Author

This will bring us closer to upgrading to LLVM 4.0 in #37609

@dylanmckay dylanmckay changed the title Update rustllvm to understand the new DIBulder::createBasicType API [LLVM 4.0] Update rustllvm to understand the new DIBulder::createBasicType API Dec 11, 2016
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Dec 11, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit f65552f has been approved by alexcrichton

@shepmaster

shepmaster commented Dec 12, 2016

Copy link
Copy Markdown
Member

FWIW, I was trying to move the alignment argument: avr-rust@350ff59

Also, trailing commas aren't supported in C function calls, IIRC. That's not this problem.

@dylanmckay

Copy link
Copy Markdown
Contributor Author

@shepmaster I didn't notice that commit earlier! That's a much better solution. I'll close this PR.

@dylanmckay dylanmckay closed this Dec 12, 2016
@shepmaster

Copy link
Copy Markdown
Member

Opened #38317 with that commit.

@dylanmckay dylanmckay deleted the llvm-4.0-createbasictype branch February 4, 2017 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants