Skip to content

improve compiletest output for errors from mir-opt tests #45722

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
merged 1 commit into from
Nov 4, 2017
Merged

improve compiletest output for errors from mir-opt tests #45722

merged 1 commit into from
Nov 4, 2017

Conversation

mikhail-m1
Copy link
Contributor

improvement:

  1. Report filename against general cannot open error
  2. Report current MIR block

@rust-highfive
Copy link
Contributor

r? @aturon

(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 Nov 2, 2017
@arielb1
Copy link
Contributor

arielb1 commented Nov 2, 2017

r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Collaborator

bors commented Nov 2, 2017

📌 Commit d6dfec1 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Nov 3, 2017

⌛ Testing commit d6dfec1 with merge 0bce08b3a89221b17c5f262b07cb0d8b590403b6...

@bors
Copy link
Collaborator

bors commented Nov 3, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Nov 3, 2017

@bors retry

kennytm added a commit to kennytm/rust that referenced this pull request Nov 4, 2017
@kennytm
Rollup merge of rust-lang#45722 - mikhail-m1:improve-mir-opt-error-ou…
7a766a4 
…tput, r=alexcrichton

improve compiletest output for errors from mir-opt tests

improvement:
1. Report filename against general cannot open error
2. Report current MIR block
@kennytm kennytm mentioned this pull request Nov 4, 2017
Merged
bors added a commit that referenced this pull request Nov 4, 2017
@bors
Auto merge of #45755 - kennytm:rollup, r=kennytm
9acc333 
Rollup of 9 pull requests

- Successful merges: #45548, #45610, #45639, #45669, #45681, #45718, #45722, #45739, #45746
- Failed merges:
@bors bors merged commit d6dfec1 into rust-lang:master Nov 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@aturon aturon

Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@mikhail-m1 @rust-highfive @arielb1 @alexcrichton @bors @kennytm @aturon