Skip to content

do not normalize non-scalar constants to a ConstValue::ScalarPair - #54759

Closed
RalfJung wants to merge 1 commit into
rust-lang:betafrom
RalfJung:ctfe-scalar-pair-undef
Closed

do not normalize non-scalar constants to a ConstValue::ScalarPair#54759
RalfJung wants to merge 1 commit into
rust-lang:betafrom
RalfJung:ctfe-scalar-pair-undef

Conversation

@RalfJung

Copy link
Copy Markdown
Member

Backport of #54693, avoiding changing ConstValue to keep the churn down.

Fixes#54387

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @estebank

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

@rust-highfive

Copy link
Copy Markdown
Contributor

⚠️Warning⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 2, 2018
@RalfJungRalfJung added beta-nominated Nominated for backporting to the compiler in the beta channel. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 2, 2018
@emilyalbiniemilyalbini added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Oct 2, 2018
@pnkfelix

Copy link
Copy Markdown
Contributor

discussed at compiler team meeting. Accepted for beta backport

@pnkfelixpnkfelix added beta-nominated Nominated for backporting to the compiler in the beta channel. beta-accepted Accepted for backporting to the compiler in the beta channel. and removed S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Oct 4, 2018
@emilyalbiniemilyalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Oct 4, 2018
@emilyalbiniemilyalbini mentioned this pull request Oct 4, 2018
@emilyalbini

Copy link
Copy Markdown
Member

Rolled this up in #54821.

bors added a commit that referenced this pull request Oct 4, 2018
[beta] Rollup backports
Merged and approved:
* #54605: resolve: Disambiguate a subset of conflicts "macro_rules" vs "macro name in module"
* #54557: incr.comp.: Don't automatically enable -Zshare-generics for incr. comp. builds.
* #54759: do not normalize non-scalar constants to a ConstValue::ScalarPair
Closes#54759
r? @ghost
@emilyalbini

Copy link
Copy Markdown
Member

This was backported in #54821.

@RalfJung
RalfJung deleted the ctfe-scalar-pair-undef branch November 30, 2018 08:30
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-acceptedAccepted for backporting to the compiler in the beta channel.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RalfJung@rust-highfive@pnkfelix@emilyalbini@estebank