Skip to content
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

Audit C++ types in rustllvm #35174

Merged
merged 5 commits into from
Aug 3, 2016
Merged

Audit C++ types in rustllvm #35174

merged 5 commits into from
Aug 3, 2016

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Aug 1, 2016

cc @eddyb

Fixes #35131

@rust-highfive
Copy link
Collaborator

r? @pnkfelix

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

@arielb1 arielb1 changed the title [WIP] audit C++ types in rustllvm Audit C++ types in rustllvm Aug 2, 2016
@eddyb
Copy link
Member

eddyb commented Aug 2, 2016

@bors r+

@bors
Copy link
Contributor

bors commented Aug 2, 2016

📌 Commit 938fb79 has been approved by eddyb

@bors
Copy link
Contributor

bors commented Aug 3, 2016

⌛ Testing commit 938fb79 with merge ff34c6b...

@bors
Copy link
Contributor

bors commented Aug 3, 2016

💔 Test failed - auto-linux-64-cross-freebsd

@arielb1
Copy link
Contributor Author

arielb1 commented Aug 3, 2016

@bors r=eddyb

@bors
Copy link
Contributor

bors commented Aug 3, 2016

📌 Commit 3041a97 has been approved by eddyb

@bors
Copy link
Contributor

bors commented Aug 3, 2016

⌛ Testing commit 3041a97 with merge a0b4e67...

bors added a commit that referenced this pull request Aug 3, 2016
@badboy
Copy link
Member

badboy commented Aug 3, 2016

Oh nice! Thanks, @arielb1

@murarth
Copy link
Contributor

murarth commented Aug 4, 2016

I don't understand how you can say that the ExecutionEngine API may bitrot when it's being used by Rusti, an actively maintained project. You can see for yourself all the "Fix for nightly" commits in its history, which goes back over a year and a half.

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 12, 2020
Added in rust-lang#35174, this was already unused (and new uses have not been introduced
since then).
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 12, 2020
Added in rust-lang#35174, this was already unused (and new uses have not been introduced
since then).
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 12, 2020
Also introduced in rust-lang#35174, and immediately unused.
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Aug 12, 2020
Also unused since introduction in rust-lang#35174
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.

7 participants