-
-
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: Update LLVM to 19.1.0 #25035
base: master
Are you sure you want to change the base?
Conversation
3 fixes here come from better diagnostics in the latest Clang.
crt0 is never compiled with this macro on in the first place.
This was a leftover from the times when CMake didn't support SerenityOS natively.
In addition to being rebased, LLVM driver no longer ever tries to link crti.o, crtn.o, or crt0_shared.o. The code that handles -pie / -static / -shared flags has been simplified as per MaskRay's suggestion (I made changes similar to llvm/llvm-project@ae623d16). Tests have been updated to match the current behavior of the driver. Note, however, that I didn't run them with non-default CLANG_DEFAULT_* options. The commit also removes Daniel's -fvisibility-inlines-hidden-function-templates patch since this functionality was never actually used and I feel like such features should either live in upstream or be deleted.
These files are always useless and are completely unused when our Clang driver is used to compile Serenity.
I looked through this and the only question I have is whether the patches were generated in the same way as the previous version? I think when I was hacking on this I would always use the same |
I think there's formatting difference because I messed up last time. I did |
No description provided.