Skip to content
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

freebsd: Register JIT unwind info #39581

Merged
merged 1 commit into from
Feb 9, 2021
Merged

freebsd: Register JIT unwind info #39581

merged 1 commit into from
Feb 9, 2021

Conversation

Keno
Copy link
Member

@Keno Keno commented Feb 9, 2021

I noticed in #39553 that JIT unwind info didn't actually
work on FreeBSD. As a general policy, we should register our frame info to make sure
that our unwind is accurate. Libunwind has some heuristics to fall back to, but they
can break e.g. for large stack frames.

I noticed in #39553 that JIT unwind info didn't actually
work on FreeBSD. As a general policy, we should register our frame info to make sure
that our unwind is accurate. Libunwind has some heuristics to fall back to, but they
can break e.g. for large stack frames.
@Keno Keno requested review from vtjnash and ararslan February 9, 2021 03:05
Copy link
Member

@ararslan ararslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks right to me. I'm assuming the CI failures across the board are unrelated...?

@ararslan ararslan added the system:freebsd Affects only FreeBSD label Feb 9, 2021
@vtjnash vtjnash merged commit 8f60d5c into master Feb 9, 2021
@vtjnash vtjnash deleted the kf/freebsdjitunwind branch February 9, 2021 20:25
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
I noticed in JuliaLang#39553 that JIT unwind info didn't actually
work on FreeBSD. As a general policy, we should register our frame info to make sure
that our unwind is accurate. Libunwind has some heuristics to fall back to, but they
can break e.g. for large stack frames.
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
I noticed in JuliaLang#39553 that JIT unwind info didn't actually
work on FreeBSD. As a general policy, we should register our frame info to make sure
that our unwind is accurate. Libunwind has some heuristics to fall back to, but they
can break e.g. for large stack frames.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system:freebsd Affects only FreeBSD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants