-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Print ParamTy
and ParamConst
instead of displaying them
#94279
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
Conversation
Display for `ParamTy` and `ParamConst` is implemented in terms of print. Using print avoids creating a new `FmtPrinter` just to display the parameter name.
@bors r+ rollup=never |
📌 Commit eaf4c91 has been approved by |
⌛ Testing commit eaf4c91 with merge 964119a9fa2ec97c2e316ad7ce59307d0556ee01... |
💔 Test failed - checks-actions |
@bors retry error: failed to download from |
☀️ Test successful - checks-actions |
Finished benchmarking commit (6cbc6c3): comparison url. Summary: This benchmark run did not return any relevant results. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
Display for
ParamTy
andParamConst
is implemented in terms of print.Using print avoids creating a new
FmtPrinter
just to display theparameter name.
r? @Mark-Simulacrum