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

Rollup of 20 pull requests #49317

Closed
wants to merge 83 commits into from
Closed

Conversation

Zoxc and others added 30 commits March 16, 2018 11:25
This commit tweaks where timing and step information is printed out as part of
the build, ensuring that we do it as close to the location where work happens as
possible. In rustbuild various functions may perform long blocking work as
dependencies are assembled, so if we print out timing information early on we
may accidentally time more than just the step we were intending to time!
LLVM seems to still put the assume calls in when inlining, so this probably isn't in a place where it can be turned on by default, but it's interesting to experiment with.

For example, this makes `swap::<u64x8>` be 8x `vmovaps	ymm` instead of 16x `vmovups	xmm`, on my cpu.
This stabilizes `main` with non-() return types; see rust-lang#48453.
@alexcrichton
Copy link
Member Author

@bors: r+ p=20

@bors
Copy link
Contributor

bors commented Mar 24, 2018

📌 Commit 455e542 has been approved by alexcrichton

@bors bors 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 Mar 24, 2018
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Mar 24, 2018

📌 Commit 95b5665 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Mar 24, 2018

⌛ Testing commit 95b5665f6c2fa7a9e43c881002cbabd05e9add9f with merge d12349a8da744aab8571247d06459bf282cd7035...

@bors
Copy link
Contributor

bors commented Mar 24, 2018

💔 Test failed - status-appveyor

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 24, 2018
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented Mar 24, 2018

📌 Commit fc23686 has been approved by alexcrichton

@bors bors 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 Mar 24, 2018
@bors
Copy link
Contributor

bors commented Mar 24, 2018

⌛ Testing commit fc23686 with merge 4accd60d3ffc908e05f3f3c2abb65b1be8172283...

@bors
Copy link
Contributor

bors commented Mar 24, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 24, 2018
@kennytm
Copy link
Member

kennytm commented Mar 24, 2018

#49046 is already testing. Creating a new rollup.

@kennytm kennytm closed this Mar 24, 2018
@alexcrichton alexcrichton deleted the rollup branch March 24, 2018 13:59
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.