yurydelendik edited PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [ ] some externref / gc tests stopped working, due to stackmaps?
yurydelendik edited PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
yurydelendik edited PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
bjorn3 submitted PR Review.
bjorn3 submitted PR Review.
bjorn3 created PR Review Comment:
This trait is public so conditionally adding this without default would break outside implementors of this trait that expect to have
unwind
disabled when it isn't due to another crate.
bjorn3 created PR Review Comment:
Maybe introduce
struct InsnRange(InsnIndex, InsnIndex);
?
bjorn3 created PR Review Comment:
All fields are public, so conditionally adding a field is a breaking change.
yurydelendik created PR Review Comment:
I would like to remove "unwind" feature. @bjorn3, as the library user, is unwind info something you are planning to use?
yurydelendik edited PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik submitted PR Review.
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [ ] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
bjorn3 submitted PR Review.
bjorn3 created PR Review Comment:
I use unwind info for
.eh_frame
generation in cg_clif.
yurydelendik created PR Review Comment:
does it make sense to remove
#[cfg(feature = "unwind")]
at this point?
yurydelendik submitted PR Review.
bjorn3 submitted PR Review.
bjorn3 created PR Review Comment:
It pulls in gimli and some users may not use unwind info. I don't know if
#[cfg(feature = "unwind")]
should be kept or removed.
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik submitted PR Review.
yurydelendik created PR Review Comment:
Addressed it in d807af3 by keeping this member mandatory to implement.
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [ ] not to use
UnwindInfoKind
?- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [ ] make unwind info available without feature?
yurydelendik edited PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
yurydelendik has marked PR #2266 as ready for review.
yurydelendik requested peterhuene for a review on PR #2266.
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
peterhuene submitted PR Review.
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
yurydelendik updated PR #2266 from new-backend-unwind
to main
:
Addresses unwind for experimental x64 item from #2079 . Code was just changed copy for the old x86 backend for SystemV. The preliminary code enables backtrace on SystemV call convension; enables "traps::*" tests there.
Investigate or not sure if possible:
- [x] fix unit test in new "systemv.rs"
- [x] detect "sp += " + "group" push
- [x] <del>not to use
UnwindInfoKind
?</del>- [x] <del>some externref / gc tests stopped working, due to stackmaps?</del>
- [x] <del>make unwind info available without feature?</del>
yurydelendik merged PR #2266.
Last updated: Dec 23 2024 at 12:05 UTC