-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
compiler-rt fails to build on NetBSD #29977
Comments
Ah this may have just been missed out in the compiler-rt update, it should be fine to basically pull in whatever the same fix was previously again. |
The correct unwind.h is also available via the libunwind package from pkgsrc. But that may or may not be a good solution. |
Looked at this more. https://nxr.netbsd.org/xref/src/lib/libc/compiler_rt/Makefile.inc?r=1.30#29 and https://nxr.netbsd.org/xref/src/share/mk/bsd.own.mk?r=1.938#127 shows that gcc_personality_v0.c is only used on EABI ARM machines, which rustc for NetBSD does not currently target. I'll probably have a patch to address that just forces HAVE_UNWIND_H=OFF on NetBSD/non-ARM. |
I believe this is no longer being built, so closing. |
With the compiler-rt update in PR #29478,
gcc_personality_v0.c
now includes theunwind.h
header to deduplicate some type definitions (commit: rust-lang/compiler-rt@5f0c79f) .The
unwind.h
header available in NetBSD however does not provide all required definitions. Specifically,_Unwind_Action
is absent in this header, leading to the following build failure:Side note: As far as I understand, NetBSD provides the above header only if the platform uses libgcc_eh (e.g. on amd64 using gcc, which is supported by Rust currently). Otherwise, LLVM's libunwind header is installed, which contains the required definitions.
I'm not sure how to fix this. Should Rust's compiler-rt duplicate the type definitions again?
The text was updated successfully, but these errors were encountered: