Support signal frame unwinding on AArch64 - #348
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #348 +/- ##
=======================================
Coverage 77.56% 77.56%
=======================================
Files 57 57
Lines 6611 6611
Branches 630 630
=======================================
Hits 5128 5128
Misses 1483 1483
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
OK, this looks like a small enough patch that I'd be willing to carry it in pystack for now, until this is addressed in elfutils proper. @K0u1e would you mind amending your commit to add a Signed-off-by: line with your full name? See https://github.com/bloomberg/pystack/blob/main/CONTRIBUTING.md#contribution-licensing - we need that as an attestation that this code is yours, and that you're willing to offer it under the terms of the Apache 2 license we use. If you amend your commit and push it up with that, I can take care of rebasing it and getting it past our lint checks. |
a46452e to
4f23f2d
Compare
Signed-Off-By: Chunyang Wang <wangchunyang.26@bytedance.com>
4f23f2d to
907cfe2
Compare
#341
Describe your changes
add patch for elfutils to unwind signal frame on aarch64
Testing performed
run test in issue and see
Additional context
https://codebrowser.dev/llvm/libunwind/src/UnwindCursor.hpp.html#2810