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

Support libunwind #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support libunwind #2

wants to merge 1 commit into from

Conversation

syu-nya
Copy link

@syu-nya syu-nya commented Sep 24, 2022

Signed-off-by: Sheng Yu [email protected]

Signed-off-by: Sheng Yu <[email protected]>
@thesamesam
Copy link
Member

@mgorny

@mgorny
Copy link
Member

mgorny commented Jan 5, 2023

Don't you need explicit -lunwind sometimes though? Perhaps AC_SEARCH_LIBS for one of the functions would be appropriate.

@syu-nya
Copy link
Author

syu-nya commented Jan 5, 2023

You mean use AC_SEARCH_LIBS for libunwind and not the header?

@mgorny
Copy link
Member

mgorny commented Jan 6, 2023

You mean use AC_SEARCH_LIBS for libunwind and not the header?

Nah, both. Use header to determine if it's available and libs to see if you need to link anything for it to work (I suspect clang + compiler-rt will require libunwind).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants