Skip to content

Rollup of 6 pull requests - #69030

Merged
bors merged 27 commits into
rust-lang:masterfrom
Dylan-DPC-zz:rollup-t9uk7vc
Feb 11, 2020
Merged

Rollup of 6 pull requests#69030
bors merged 27 commits into
rust-lang:masterfrom
Dylan-DPC-zz:rollup-t9uk7vc

Conversation

@Dylan-DPC-zz

Copy link
Copy Markdown

Successful merges:

Failed merges:

r? @ghost

jwhite927and others added 27 commits February 5, 2020 06:28
Add long error code explanation message for E0637
Reference issue [rust-lang#61137](rust-lang#61137)
To incorporate a long error description for E0637, I have made the necessary modification to error_codes.rs and added error_codes/E0637.md, and blessed the relevant .stderror files. ~~, however when I build rustc stage 1, I am unable to make `$ rustc --explain E0637` work even though rustc appears to be able to call up the long error explanations for other errors. I wanted to guarantee this would work before moving on the blessing the various ui tests that have been affected. @GuillaumeGomez Do you know the most likely reason(s) why this would be the case?~~
Update: `$ rustc --explain E0637` works now.
…ic-activity-args, r=wesleywiser
self-profile: Support arguments for generic_activities.
This PR adds support for recording arguments of "generic activities". The most notable use case is LLVM module names, which should be very interesting for `crox` profiles. In the future it might be interesting to add more fine-grained events for pre-query passes like macro expansion.
I tried to judiciously de-duplicate existing self-profile events with `extra_verbose_generic_activity`, now that the latter also generates self-profile events.
r? @wesleywiser
…=Centril
Make ASCII ctype functions unstably const
Makes the following inherent methods on `u8` and `char` unstable `const fn`:
* `is_ascii_alphabetic`
* `is_ascii_uppercase`
* `is_ascii_lowercase`
* `is_ascii_alphanumeric`
* `is_ascii_digit`
* `is_ascii_hexdigit`
* `is_ascii_punctuation`
* `is_ascii_graphic`
* `is_ascii_whitespace`
* `is_ascii_control`
cc rust-lang#68983
change an instance of span_bug() to struct_span_err() to avoid ICE
After rust-lang#67148, the `span_bug()` in `parse_ty_tuple_or_parens()` is reachable because `parse_paren_comma_seq()` can return an `Ok()` even in cases where it encounters an error.
This pull request prevents an ICE in such cases by replacing the `span_bug()` with `struct_span_error()`.
Fixesrust-lang#68890.
@Dylan-DPC-zz

Copy link
Copy Markdown
Author

@bors r+ rollup=never p=6

@bors

bors commented Feb 10, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 119bc97 has been approved by Dylan-DPC

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 10, 2020
@Dylan-DPC-zzDylan-DPC-zz added the rollup A PR which is a rollup label Feb 10, 2020
@bors

bors commented Feb 10, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 119bc97 with merge 3097a84...

bors added a commit that referenced this pull request Feb 10, 2020
Rollup of 6 pull requests
Successful merges:
- #68897 (clean up E0275 explanation)
- #68908 (Add long error code explanation message for E0637 )
- #68932 (self-profile: Support arguments for generic_activities.)
- #68986 (Make ASCII ctype functions unstably const )
- #69007 (Clean up E0283 explanation)
- #69014 (change an instance of span_bug() to struct_span_err() to avoid ICE)
Failed merges:
r? @ghost
@bors

bors commented Feb 10, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 119bc97 with merge 0f0cdf6...

bors added a commit that referenced this pull request Feb 10, 2020
Rollup of 6 pull requests
Successful merges:
- #68897 (clean up E0275 explanation)
- #68908 (Add long error code explanation message for E0637 )
- #68932 (self-profile: Support arguments for generic_activities.)
- #68986 (Make ASCII ctype functions unstably const )
- #69007 (Clean up E0283 explanation)
- #69014 (change an instance of span_bug() to struct_span_err() to avoid ICE)
Failed merges:
r? @ghost
@bors

bors commented Feb 11, 2020

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-azure
Approved by: Dylan-DPC
Pushing 0f0cdf6 to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Feb 11, 2020
@bors
bors merged commit 119bc97 into rust-lang:masterFeb 11, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: 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.

8 participants

@Dylan-DPC-zz@bors@jwhite927@ecstatic-morse@dwrensha@GuillaumeGomez@michaelwoerister@Dylan-DPC