Skip to content
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

Better portability #66

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

GlowingScrewdriver
Copy link
Contributor

This PR incorporates changes to make the compiler framework easier to run on other systems, particularly Mac OS X.

Writing POSIX-compliant shell scripts alone should take us a long way in portability.

@chsasank
Copy link
Owner

chsasank commented Jul 8, 2024

We can consider ditching temp files and actually compile using llvmlite. Yes, LLVMLite supports that. https://llvmlite.readthedocs.io/en/latest/user-guide/binding/examples.html

@chsasank
Copy link
Owner

chsasank commented Jul 8, 2024

Problem we'll face then is linking with other C libraries. So keeping clang driver might be useful.

@GlowingScrewdriver
Copy link
Contributor Author

Problem we'll face then is linking with other C libraries. So keeping clang driver might be useful.

I agree. Continuing to use Clang will keep things convenient for us. Although making the switch won't be too tough, since Clang calls an external linker (e.g. ld from GNU Coreutils) anyways.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants