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

build llvm in //libs/jsruntime/build.rs #155

Open
masnagam opened this issue May 18, 2024 · 1 comment
Open

build llvm in //libs/jsruntime/build.rs #155

masnagam opened this issue May 18, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@masnagam
Copy link
Member

masnagam commented May 18, 2024

currently, we build llvm before running cargo commands. this was an easier way but there are the following disadvantages:

  • llvm is built w/ specific options
    • we cannot change options in accordance with the build profile to use
    • LLVM_USE_PERF, LLVM_USE_SANITIZER, etc.
  • we have to build it manually before running cargo commands

change //libs/jsruntime/build.rs and specify build options according to the build profile. we can use the cmake crate for building c++ bridge files together with llvm.

@masnagam masnagam added the enhancement New feature or request label May 18, 2024
@masnagam
Copy link
Member Author

masnagam commented May 19, 2024

this is another story, but using the shared library (libLLVM.so) may improve disk usage. currently, the binaries generated by make [build|test|bench] consume tens of GB.

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

No branches or pull requests

1 participant