Skip to content

Conversation

alexcrichton
Copy link
Member

No sense of urgency, just wanted to make sure we don't lag too far behind!

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 24, 2018
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 24, 2018

📌 Commit 97a786f has been approved by nikomatsakis

@alexcrichton
Copy link
Member Author

@bors: r-

Hm testing locally shows this is not ready. rust-lang/compiler-builtins#222 may have an accidental regression

No sense of urgency, just wanted to make sure we don't lag too far behind!
@alexcrichton alexcrichton force-pushed the update-compiler-builtins branch from 97a786f to 54462f5 Compare January 25, 2018 02:26
@alexcrichton
Copy link
Member Author

@bors: r=nikomatsakis

@bors
Copy link
Collaborator

bors commented Jan 25, 2018

📌 Commit 54462f5 has been approved by nikomatsakis

@nikomatsakis nikomatsakis added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 25, 2018
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 25, 2018
…ns, r=nikomatsakis

Update compiler-builtins submodule

No sense of urgency, just wanted to make sure we don't lag too far behind!
bors added a commit that referenced this pull request Jan 26, 2018
@bors bors merged commit 54462f5 into rust-lang:master Jan 26, 2018
@alexcrichton alexcrichton deleted the update-compiler-builtins branch January 26, 2018 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants