Fix debuginfo for parameters passed via the ScalarPair abi on Windows#88629
Conversation
|
r? @oli-obk (rust-highfive has picked a reviewer for you, use r? to override) |
oli-obk
left a comment
There was a problem hiding this comment.
When adding instrumentation to a function I usually also go through the function and update all debug! invocations to their tracing equivalent
- debug!("hello: foo = {:?}", foo);
+ debug!(?foo, "hello");Not saying you have to do this, that's just what I've been doing ^^.
|
@bors r+ |
|
📌 Commit 7bbdc884a2258f96fcff8fcea3d4d6e2acf6e294 has been approved by |
|
⌛ Testing commit 7bbdc884a2258f96fcff8fcea3d4d6e2acf6e294 with merge f60df93e93ea66543bb55346f0f264a9bb70da07... |
|
💔 Test failed - checks-actions |
This comment has been minimized.
This comment has been minimized.
7bbdc88 to
3195401
Compare
|
📌 Commit 3195401 has been approved by |
…rpair_params, r=oli-obk Fix debuginfo for parameters passed via the ScalarPair abi on Windows Mark all of these as locals so the debugger does not try to interpret them as being a pointer to the value. This extends the approach used in rust-lang#81898. Fixes rust-lang#88625
|
@bors r- |
Mark all of these as locals so the debugger does not try to interpret them as being a pointer to the value. This extends the approach used in PR rust-lang#81898.
3195401 to
a6c642a
Compare
|
Resolved type name for @bors r=oli-obk |
|
📌 Commit a6c642a has been approved by |
|
⌛ Testing commit a6c642a with merge 384ced8504e71333555a67050fff22e7eddd2df7... |
|
💥 Test timed out |
|
@bors retry |
|
☀️ Test successful - checks-actions |
|
Finished benchmarking commit (5fd6f3b): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
Mark all of these as locals so the debugger does not try to interpret
them as being a pointer to the value. This extends the approach used
in #81898.
Fixes #88625