-
Notifications
You must be signed in to change notification settings - Fork 11
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
Split off functionality that relies on libm #18
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine @andreittr, thank you.
allowing a minimal compiler-rt to be built with nolibc alone
What is the minimal setup for building compiler-rt
with this pr? I suppose libunwind
it's still needed, right?
Previously some parts of compiler-rt depended on functions provided by a math library, commonly supplied together with a real libc. This change sections off the parts dependent on libm, allowing a minimal compiler-rt to be built with nolibc alone. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
3b76b0c
to
8052e81
Compare
Nope, following the merging of #14 libunwind is no longer needed, except if you really want that gcc_personality_v0. A plainly configured also: rebased on top of staging, fixed conflicts & re-pushed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @andreittr, all good.
Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved-by: Razvan Deaconescu razvand@unikraft.io
Previously some parts of compiler-rt depended on functions provided by a math library, commonly supplied together with a real libc. This change sections off the parts dependent on libm, allowing a minimal compiler-rt to be built with nolibc alone.