diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-06-23 16:12:39 +1000 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2024-06-25 14:35:09 +1000 |
| commit | 9828e960ab4cccc9526fd6cc61df430588bd0487 (patch) | |
| tree | 26e565be478e49c3dc53ca92f66302180e6f83fb /compiler/rustc_mir_transform/src/coverage/counters.rs | |
| parent | 684b3553f70148ded97a80371c2405984d4f6aa7 (diff) | |
| download | rust-9828e960ab4cccc9526fd6cc61df430588bd0487.tar.gz rust-9828e960ab4cccc9526fd6cc61df430588bd0487.zip | |
Remove `__rust_force_expr`.
This was added (with a different name) to improve an error message. It is no longer needed -- removing it changes the error message, but overall I think the new message is no worse: - the mention of `#` in the first line is a little worse, - but the extra context makes it very clear what the problem is, perhaps even clearer than the old message, - and the removal of the note about the `expr` fragment (an internal detail of `__rust_force_expr`) is an improvement. Overall I think the error is quite clear and still far better than the old message that prompted #61933, which didn't even mention patterns. The motivation for this is #124141, which will cause pasted metavariables to be tokenized and reparsed instead of the AST node being cached. This change in behaviour occasionally has a non-zero perf cost, and `__rust_force_expr` causes the tokenize/reparse step to occur twice. Removing `__rust_force_expr` greatly reduces the extra overhead for the `deep-vector` benchmark.
Diffstat (limited to 'compiler/rustc_mir_transform/src/coverage/counters.rs')
0 files changed, 0 insertions, 0 deletions
