Skip to content

ir: Cleanup a bunch of constructors #1095

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

Merged
merged 2 commits into from
Oct 24, 2017
Merged

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Oct 24, 2017

Should be no change in behavior.

jrmuizel and others added 2 commits October 24, 2017 08:59
This lets us capture 'static inline' functions in the ir and filter them
later down the pipeline. This is the first step on the way to handling
these functions better.
To use the shorthand initialization syntax.
@emilio
Copy link
Contributor Author

emilio commented Oct 24, 2017

This is on top of #1091 because it touches one of them.

r? @pepyakin or @fitzgen

@pepyakin
Copy link
Contributor

Looks good to me!

@bors-servo r+

@bors-servo
Copy link

📌 Commit 3c4ccd2 has been approved by pepyakin

@bors-servo
Copy link

⌛ Testing commit 3c4ccd2 with merge 1831867...

bors-servo pushed a commit that referenced this pull request Oct 24, 2017
ir: Cleanup a bunch of constructors

Should be no change in behavior.
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: pepyakin
Pushing 1831867 to master...

@bors-servo bors-servo merged commit 3c4ccd2 into rust-lang:master Oct 24, 2017
@emilio emilio deleted the nits branch October 24, 2017 09:15
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