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

make mem::discriminant const #69825

Merged
merged 6 commits into from
Mar 11, 2020
Merged

make mem::discriminant const #69825

merged 6 commits into from
Mar 11, 2020

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented Mar 8, 2020

implements #69821, which could be used as a tracking issue for const_discriminant.

Should this be added to the meta tracking issue #57563?
@Lokathor

@rust-highfive
Copy link
Collaborator

r? @eddyb

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 8, 2020
@Centril
Copy link
Contributor

Centril commented Mar 8, 2020

r? @RalfJung

@rust-highfive rust-highfive assigned RalfJung and unassigned eddyb Mar 8, 2020
@RalfJung
Copy link
Member

RalfJung commented Mar 8, 2020

Sure, but also Cc @rust-lang/wg-const-eval FYI

@rust-highfive

This comment has been minimized.

@Lokathor Lokathor mentioned this pull request Mar 8, 2020
25 tasks
@rust-highfive

This comment has been minimized.

@oli-obk
Copy link
Contributor

oli-obk commented Mar 10, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Mar 10, 2020

📌 Commit 7b3e3ff has been approved by oli-obk

@bors
Copy link
Contributor

bors commented Mar 10, 2020

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@bors bors removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 10, 2020
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 10, 2020
Centril added a commit to Centril/rust that referenced this pull request Mar 11, 2020
make `mem::discriminant` const

implements rust-lang#69821, which could be used as a tracking issue for `const_discriminant`.

Should this be added to the meta tracking issue rust-lang#57563?
@Lokathor
bors added a commit that referenced this pull request Mar 11, 2020
Rollup of 10 pull requests

Successful merges:

 - #66059 (mem::zeroed/uninit: panic on types that do not permit zero-initialization)
 - #69373 (Stabilize const for integer {to,from}_{be,le,ne}_bytes methods)
 - #69591 (Use TypeRelating for instantiating query responses)
 - #69625 (Implement nth, last, and count for iter::Copied)
 - #69645 (const forget tests)
 - #69766 (Make Point `Copy` in arithmetic documentation)
 - #69825 (make `mem::discriminant` const)
 - #69859 (fix #62456)
 - #69891 (Exhaustiveness checking, `Matrix::push`: recursively expand or-patterns)
 - #69896 (parse: Tweak the function parameter edition check)

Failed merges:

r? @ghost
@bors bors merged commit dfbbd5d into rust-lang:master Mar 11, 2020
@lcnr lcnr deleted the discriminant branch March 11, 2020 13:35
bors added a commit to rust-lang/miri that referenced this pull request Mar 12, 2020
rm custom `intrinsics::discriminant_value`

@RalfJung requires rust-lang/rust#69825
bors added a commit to rust-lang/miri that referenced this pull request Mar 12, 2020
rm custom `intrinsics::discriminant_value`

@RalfJung requires rust-lang/rust#69825
bors added a commit to rust-lang/miri that referenced this pull request Mar 12, 2020
rm custom `intrinsics::discriminant_value`

@RalfJung requires rust-lang/rust#69825
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.

7 participants