-
Notifications
You must be signed in to change notification settings - Fork 35
Out-Of-Process Interpreter for CppInterOp #717
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?
Changes from 16 commits
460fc97
72fee63
d77c457
6742433
6cf1da4
7217df9
334b02e
3d9669d
ab50e04
c9a4426
9e18da6
748d7c0
bf2637e
f06dcc8
9250d0f
cfbb303
b32b7b8
d57d112
b6177a8
85da62f
f330c78
a3fd0a9
609f7a4
62443bb
18b5b3c
5621046
e2b35ec
1d307ff
3580f96
8829b88
02f0f4f
5296a33
ad47408
f5f7fef
30c89be
a4b186a
1e3fd0f
eb34615
f047650
fdd3dfb
1a3d3d6
93c7441
6c26160
ee93b0e
7cb45d2
dbb1cdc
7765e67
a2a3dce
acf6ebf
94b9a8e
a615546
a0fdb40
c34f786
fd3ccf5
3828d54
e9a2ddd
86c79ba
9da822f
50e78b3
8510a89
e45fb0c
509a931
e2e6702
0f2c944
612af09
b5c488b
c041faf
c900d0b
d03a754
2b3fcee
4a2c91e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,15 +23,6 @@ jobs: | |
| matrix: | ||
| include: | ||
| # Ubuntu Arm Jobs | ||
| - name: ubu22-arm-gcc12-clang-repl-20-coverage | ||
| os: ubuntu-22.04-arm | ||
| compiler: gcc-12 | ||
| clang-runtime: '20' | ||
| cling: Off | ||
| cppyy: Off | ||
| llvm_enable_projects: "clang" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| coverage: true | ||
| - name: ubu24-arm-gcc12-clang-repl-20 | ||
| os: ubuntu-24.04-arm | ||
| compiler: gcc-12 | ||
|
|
@@ -66,6 +57,16 @@ jobs: | |
| llvm_enable_projects: "clang" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| # Ubuntu X86 Jobs | ||
| - name: ubu22-x86-gcc12-clang-repl-20-coverage | ||
|
||
| os: ubuntu-22.04 | ||
| compiler: gcc-12 | ||
| clang-runtime: '20' | ||
| cling: Off | ||
| cppyy: Off | ||
| llvm_enable_projects: "clang;compiler-rt" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| coverage: true | ||
| oop-jit: On | ||
| - name: ubu24-x86-gcc12-clang-repl-20 | ||
| os: ubuntu-24.04 | ||
| compiler: gcc-12 | ||
|
|
@@ -74,6 +75,15 @@ jobs: | |
| cppyy: Off | ||
| llvm_enable_projects: "clang" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| - name: ubu24-x86-gcc12-clang-repl-20-out-of-process | ||
| os: ubuntu-24.04 | ||
| compiler: gcc-12 | ||
| clang-runtime: '20' | ||
| cling: Off | ||
| cppyy: Off | ||
| llvm_enable_projects: "clang;compiler-rt" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| oop-jit: On | ||
| - name: ubu24-x86-gcc12-clang-repl-19-cppyy | ||
| os: ubuntu-24.04 | ||
| compiler: gcc-12 | ||
|
|
@@ -100,6 +110,15 @@ jobs: | |
| llvm_enable_projects: "clang" | ||
| llvm_targets_to_build: "host;NVPTX" | ||
| # MacOS Arm Jobs | ||
| - name: osx15-arm-clang-clang-repl-20-out-of-process | ||
| os: macos-15 | ||
| compiler: clang | ||
| clang-runtime: '20' | ||
| cling: Off | ||
| cppyy: Off | ||
| llvm_enable_projects: "clang;compiler-rt" | ||
| llvm_targets_to_build: "host" | ||
| oop-jit: On | ||
| - name: osx15-arm-clang-clang-repl-20 | ||
| os: macos-15 | ||
| compiler: clang | ||
|
|
@@ -220,7 +239,7 @@ jobs: | |
| path: | | ||
| llvm-project | ||
| ${{ matrix.cling=='On' && 'cling' || '' }} | ||
| key: ${{ env.CLING_HASH }}-${{ runner.os }}-${{ matrix.os }}-${{ matrix.compiler }}-clang-${{ matrix.clang-runtime }}.x-patch-${{ hashFiles(format('patches/llvm/clang{0}-*.patch', matrix.clang-runtime)) || 'none' }} | ||
| key: ${{ env.CLING_HASH }}-${{ runner.os }}-${{ matrix.os }}-${{ matrix.compiler }}-clang-${{ matrix.clang-runtime }}.x-patch-${{ hashFiles(format('patches/llvm/clang{0}-*.patch', matrix.clang-runtime)) || 'none' }}${{ matrix.oop-jit == 'On' && '-oop' || '' }} | ||
|
|
||
| - name: Setup default Build Type | ||
| uses: ./.github/actions/Miscellaneous/Select_Default_Build_Type | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -298,6 +298,18 @@ include_directories(SYSTEM ${LLVM_INCLUDE_DIRS}) | |
| separate_arguments(LLVM_DEFINITIONS_LIST NATIVE_COMMAND ${LLVM_DEFINITIONS}) | ||
| add_definitions(${LLVM_DEFINITIONS_LIST}) | ||
|
|
||
| string(REGEX REPLACE "/lib/cmake/llvm$" "" LLVM_BUILD_DIR "${LLVM_DIR}") | ||
| add_definitions(-DLLVM_BUILD_DIR="${LLVM_BUILD_DIR}") | ||
|
|
||
| if(LLVM_BUILT_WITH_OOP_JIT) | ||
| if((CMAKE_SYSTEM_NAME STREQUAL "Darwin" AND CMAKE_SYSTEM_PROCESSOR MATCHES "arm64") OR | ||
| (CMAKE_SYSTEM_NAME STREQUAL "Linux" AND CMAKE_SYSTEM_PROCESSOR MATCHES "x86_64")) | ||
| add_definitions(-DLLVM_BUILT_WITH_OOP_JIT) | ||
| else() | ||
| message(FATAL_ERROR "LLVM_BUILT_WITH_OOP_JIT is only supported on Darwin arm64 or Linux x86_64. Build aborted.") | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would switch Darwin in this text to MacOS. |
||
| endif() | ||
| endif() | ||
|
|
||
| # If the llvm sources are present add them with higher priority. | ||
| if (LLVM_BUILD_MAIN_SRC_DIR) | ||
| # LLVM_INCLUDE_DIRS contains the include paths to both LLVM's source and | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although you will see it in other parts of the ci, please remove
-j ${{ env.ncpus }}. It shouldn't be in the other parts of the ci. Ninja defaults to a higher number of processes than the number of cpu cores.