Skip to content

Add test for issue #84957 - str.as_bytes() in a const expression - #89997

Merged
bors merged 1 commit into
rust-lang:masterfrom
cameron1024:const-str-as-bytes-ice
Oct 19, 2021
Merged

Add test for issue #84957 - str.as_bytes() in a const expression#89997
bors merged 1 commit into
rust-lang:masterfrom
cameron1024:const-str-as-bytes-ice

Conversation

@cameron1024

@cameron1024cameron1024 commented Oct 18, 2021

Copy link
Copy Markdown
Contributor

Hi, this PR adds a test for issue #84957 . I'm quite new to rustc so let me know if there's anything else that needs doing 😄

Closes#84957

@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 18, 2021
@cameron1024cameron1024 changed the title added test issue 84957Add test for issue #84957 - str.as_bytes() in a const expressionOct 18, 2021
@rust-log-analyzer

This comment has been minimized.

@JohnTitor

Copy link
Copy Markdown
Member

Could you rename the file to issue-84957-const-str-as-bytes.rs and squash commits into one?
r? @JohnTitor

@cameron1024
cameron1024force-pushed the const-str-as-bytes-ice branch from a223a3a to f8b2f91CompareOctober 18, 2021 07:41
@JohnTitor

Copy link
Copy Markdown
Member

Thanks!
@bors r+ rollup=always

@bors

bors commented Oct 18, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit f8b2f91 has been approved by JohnTitor

@borsbors 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 Oct 18, 2021
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 18, 2021
… r=JohnTitor
Add test for issue rust-lang#84957 - `str.as_bytes()` in a `const` expression
Hi, this PR adds a test for issue rust-lang#84957 . I'm quite new to rustc so let me know if there's anything else that needs doing 😄
Closesrust-lang#84957
This was referenced Oct 18, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 19, 2021
…askrgr
Rollup of 8 pull requests
Successful merges:
- rust-lang#89766 (RustWrapper: adapt for an LLVM API change)
- rust-lang#89867 (Fix macro_rules! duplication when reexported in the same module)
- rust-lang#89941 (removing TLS support in x86_64-unknown-none-hermitkernel)
- rust-lang#89956 (Suggest a case insensitive match name regardless of levenshtein distance)
- rust-lang#89988 (Do not promote values with const drop that need to be dropped)
- rust-lang#89997 (Add test for issue rust-lang#84957 - `str.as_bytes()` in a `const` expression)
- rust-lang#90002 (:arrow_up: rust-analyzer)
- rust-lang#90034 (Tiny tweak to Iterator::unzip() doc comment example.)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit c3cfa58 into rust-lang:masterOct 19, 2021
@rustbotrustbot added this to the 1.58.0 milestone Oct 19, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-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.

Panic in compiler\rustc_mir\src\borrow_check\region_infer\mod.rs:2120:35

7 participants

@cameron1024@rust-highfive@rust-log-analyzer@JohnTitor@bors@Mark-Simulacrum@rustbot