-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Allow locals and destructuring in const fn #2341
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,68 @@ | ||
- Feature Name: const_locals | ||
- Start Date: 2018-01-11 | ||
- RFC PR: (leave this empty) | ||
- Rust Issue: (leave this empty) | ||
|
||
# Summary | ||
[summary]: #summary | ||
|
||
Allow `let` bindings in the body of constants and const fns. Additionally enable | ||
destructuring in `let` bindings and const fn arguments. | ||
|
||
# Motivation | ||
[motivation]: #motivation | ||
|
||
It makes writing const fns much more like writing regular functions and is | ||
not possible right now because the old constant evaluator was a constant folder | ||
that could only process expressions. With the miri const evaluator this feature | ||
exists but is still disallowed. | ||
|
||
# Guide-level explanation | ||
[guide-level-explanation]: #guide-level-explanation | ||
|
||
`let` bindings in constants and const fn work just like `let` bindings | ||
everywhere else. Historically these did not exist in constants and const fn | ||
because it would have been very hard to support them in the old const evaluator. | ||
|
||
This means that you can only move out of any let binding once, even though in a | ||
const environment obtaining a copy of the object could be done by executing the | ||
code twice, side effect free. All invariants held by runtime code are also | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
upheld by constant evaluation. | ||
|
||
# Reference-level explanation | ||
[reference-level-explanation]: #reference-level-explanation | ||
|
||
Expressions like `a + b + c` are already transformed to | ||
|
||
```rust | ||
let tmp = a + b; | ||
tmp + c | ||
``` | ||
|
||
With this RFC we can create bindings ourselves instead of only allowing compiler | ||
generated bindings. | ||
|
||
# Drawbacks | ||
[drawbacks]: #drawbacks | ||
|
||
You can create mutable locals in constants and then actually modify them. This | ||
has no real impact on the constness, as the mutation happens entirely at compile | ||
time and results in an immutable value. | ||
|
||
# Rationale and alternatives | ||
[alternatives]: #alternatives | ||
|
||
The backend already supports this 100%. This is essentially just disabling a | ||
check | ||
|
||
## Why is this design the best in the space of possible designs? | ||
|
||
Being the only design makes it the best design by definition | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🤣 |
||
|
||
## What is the impact of not doing this? | ||
|
||
Not having locals and destructuring severely limits the functions that can be | ||
turned into const fn and generally leads to unreadable const fns. | ||
|
||
# Unresolved questions | ||
[unresolved]: #unresolved-questions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let
bindings "everywhere else" (normalfn
s) introduce bindings which have move semantics whentypeof(binding) /: Copy
. Does this hold inconst fn
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. All code is compiled normally in const fn. The only difference is that the execution is on virtual hardware instead of real hardware. So if your code fails to compile outside of a const environment, it will also fail to compile within a const environment. The reverse does not hold, there's a lot of code that compiles right now that will never compile in a const environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great 👍. I'd include an explanation of this in the RFC text to make things crystal clear =)