-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Toolchain: Upstreaming tracking for gcc/binutils/llvm #14183
Comments
In order for |
I believe our locale.h is compliant -- the C library only accepts the C locale. The issue with ctype relates to us pretending to be newlib but not having the elements in the |
Turns out plenty of systems tell libc++ to provide its own "Rune table" ... so us doing so as well isn't that big a deal. |
We should update the CMake patches to reflect the fact that LibDL is now an ld-script stub like Musl's by removing the LibDL lib variable from the Platform files. |
Not sure if this is a typo, but better safe than sorry. Both LibDl and LibM are completely normal libraries at the moment, the only library that has been merged into LibC so far is LibPthread. That said, I do plan to do the same with LibDl and LibM some time in the near future. |
I created an RFC for LLVM upstreaming here: https://discourse.llvm.org/t/rfc-add-support-for-serenityos/71641 |
In order to be a fully accepted CMake target, we should have compiler support upstreamed to at least gcc or llvm-project.
Here's the list of things that need done for that do have a chance of succeeding:
This issue is being tracked at the same time on CMake's GitLab here
In SerenityOS/serenity:
Loader/Kernel/LibELF:
LibC:
<ctype.h>
POSIX and C standard compliant. Remove hax from libc++ and libstdc++ patches pretending to be newlib. (Toolchain+LibC: Remove unecessary llvm patches, add tests for ctype #14401)x86_64
GCC port actually useable #14189)GNU Autotools:
Binutils:
x86_64
GCC port actually useable #14189)GCC/libstdc++:
LLVM:
<initializer_list>
(Toolchain+LibC: Remove unecessary llvm patches, add tests for ctype #14401)LLVM Lit tests:
The text was updated successfully, but these errors were encountered: