llvm / llvm/llvm-project

[clang-repl] Drop ORC JIT as a dependecy for clangInterpreter's wasm build

Open
#175,907 2 comments 0 reactions 0 assignees View on GitHub
clang-repl
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

This has been under discussion since quite some time. What we realize is running clang-repl in the browser doesn't need ORC JIT as the wasm based incremental executor puts emscripten's dlopen mechanism to use.

As can be seen in the cmake, we still compile all of ORCJIT for wasm and link it into clangInterpreter causing uneccessary bloat.

https://github.com/llvm/llvm-project/blob/fc81a665133fe757741d68f07245d5f696a65346/clang/lib/Interpreter/CMakeLists.txt#L1-L14

I could find some ways to get past this. Spamming lot of ifdef's in itself checking if the build is for a native platform vs wasm is a way but this was a cleaner way forward.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.