Skip to content

Avoid using getDataLayout, deprecated in LLVM 3.7 #29016

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
Merged
bors merged 1 commit into from
Oct 14, 2015
Merged

Avoid using getDataLayout, deprecated in LLVM 3.7 #29016

bors merged 1 commit into from
Oct 14, 2015

Conversation

@sanxiyn
Copy link
Contributor

@sanxiyn sanxiyn commented Oct 13, 2015

No description provided.

@rust-highfive
Copy link
Contributor

rust-highfive commented Oct 13, 2015

r? @brson

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

@sanxiyn
Copy link
Contributor Author

sanxiyn commented Oct 13, 2015

Quoting include/llvm/Target/TargetMachine.h, "Deprecated in 3.7, will be removed in 3.8. Use createDataLayout() instead."

@alexcrichton
Copy link
Member

alexcrichton commented Oct 13, 2015

@bors: r+ 22dc408

@Manishearth Manishearth mentioned this pull request Oct 14, 2015
Merged
Manishearth added a commit to Manishearth/rust that referenced this pull request Oct 14, 2015
@Manishearth
Rollup merge of rust-lang#29016 - sanxiyn:data-layout, r=alexcrichton
3636c8b
bors added a commit that referenced this pull request Oct 14, 2015
@bors
Auto merge of #29039 - Manishearth:rollup, r=Manishearth
294ef5b 
- Successful merges: #28991, #29004, #29006, #29013, #29016, #29024, #29027, #29028, #29029, #29032, #29035
- Failed merges:
@bors bors merged commit 22dc408 into rust-lang:master Oct 14, 2015
@sanxiyn sanxiyn deleted the data-layout branch October 14, 2015 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@brson brson

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sanxiyn @rust-highfive @alexcrichton @brson @bors