-
Notifications
You must be signed in to change notification settings - Fork 21
feat: replace LLVM with C23 intrinsics #51
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
base: main
Are you sure you want to change the base?
Conversation
This reverts commit abd9788.
|
Nice proposal! I really like the suggestion. Do you know how widely supported |
|
Another option which I would be in favor of would be to not replace LLVM intrinsics, but instead change |
|
afaict gcc and clang have support for most common architectures, you can use the -fexperimental flag on clang to increase the max width where not supported (see build script) |
|
the build script for macOS isn't x86 comapt, brew has different paths on that chipset |
|
Hey, sorry for taking so long to get back to you on this - I couldn't find the time to look into First of all, I think this is a great idea, but there are some issues which make me not want to merge this in its current state:
|
Use
C23's_BitInt(256), which is more portable than compiling Rust to LLVM textual IR, manually replacing i128 with i256, and compiling it with clang.