Skip to content

Commit 91cb919

Browse files
committed
Get rid of the GCC gccjit backend, it got stale and doesn't provide value
Signed-off-by: Lukasz Stafiniak <lukstafi@gmail.com>
1 parent 789d7b5 commit 91cb919

File tree

11 files changed

+7
-876
lines changed

11 files changed

+7
-876
lines changed

CLAUDE.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ dune runtest -p neural_nets_lib
3030
# Install dependencies
3131
opam install . --deps-only
3232

33-
# Install with optional backends
34-
opam install cudajit gccjit # for CUDA and GCC backends
33+
# Install with optional backends
34+
opam install cudajit # for CUDA backend
3535
```
3636

3737
## Architecture Overview
@@ -66,7 +66,7 @@ opam install cudajit gccjit # for CUDA and GCC backends
6666

6767
2. **Shape Inference**: Comprehensive axis tracking with batch/input/output classification and optional dimension labels
6868

69-
3. **Backend Architecture**: Unified interface supporting CPU (multicore, GCC JIT), CUDA, and Metal backends
69+
3. **Backend Architecture**: Unified interface supporting CPU (multicore), CUDA, and Metal backends
7070

7171
4. **Memory Management**: Sophisticated tensor node memory modes (Virtual, Local, On_device, Hosted) with automatic host transfers
7272

arrayjit.opam

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ description:
88
maintainer: ["Lukasz Stafiniak <lukstafi@gmail.com>"]
99
authors: ["Lukasz Stafiniak"]
1010
license: "BSD-2-Clause"
11-
tags: ["deeplearning" "array" "jit" "gccjit" "CUDA" "Metal"]
11+
tags: ["deeplearning" "array" "jit" "CUDA" "Metal"]
1212
homepage: "https://github.com/lukstafi/ocannl"
1313
doc: "https://github.com/lukstafi/ocannl/blob/master/README.md"
1414
bug-reports: "https://github.com/lukstafi/ocannl/issues"
@@ -39,11 +39,9 @@ depends: [
3939
]
4040
depopts: [
4141
"cudajit" {>= "0.7.0"}
42-
"gccjit" {>= "0.3.2"}
4342
]
4443
conflicts: [
4544
"cudajit" {< "0.7.0"}
46-
"gccjit" {< "0.3.2"}
4745
]
4846
build: [
4947
["dune" "subst"] {dev}

arrayjit/lib/backends.ml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -618,14 +618,8 @@ let%track5_sexp fresh_backend ?backend_name ?(config = For_parallel_copying) ()
618618
| "multicore_cc" ->
619619
(module Make_device_backend_from_lowered (Schedulers.Multicore) (Cc_backend) (Config)
620620
: Backend)
621-
| "gccjit" ->
622-
(module Make_device_backend_from_lowered (Schedulers.Multicore) (Gcc_backend_impl) (Config)
623-
: Backend)
624621
| "sync_cc" ->
625622
(module Make_device_backend_from_lowered (Schedulers.Sync) (Cc_backend) (Config) : Backend)
626-
| "sync_gccjit" ->
627-
(module Make_device_backend_from_lowered (Schedulers.Sync) (Gcc_backend_impl) (Config)
628-
: Backend)
629623
| "cuda" ->
630624
(module Raise_backend ((Cuda_backend_impl.Fresh (Config) : Lowered_backend)) : Backend)
631625
| "metal" ->

arrayjit/lib/dune

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -90,21 +90,6 @@
9090
ppx_variants_conv
9191
ppx_minidebug)))
9292

93-
(library
94-
(name gcc_backend)
95-
(public_name arrayjit.gcc_backend)
96-
(optional)
97-
(modules gcc_backend)
98-
(libraries base gccjit utils ir)
99-
(preprocess
100-
(pps
101-
ppx_compare
102-
ppx_hash
103-
ppx_here
104-
ppx_sexp_conv
105-
ppx_string
106-
ppx_variants_conv
107-
ppx_minidebug)))
10893

10994
(library
11095
(name backends)
@@ -123,11 +108,6 @@
123108
saturn_lockfree
124109
utils
125110
ir
126-
(select
127-
gcc_backend_impl.ml
128-
from
129-
(gcc_backend -> gcc_backend_impl.gccjit.ml)
130-
(-> gcc_backend_impl.missing.ml))
131111
(select
132112
cuda_backend_impl.ml
133113
from
@@ -150,7 +130,6 @@
150130
ppx_minidebug))
151131
(modules
152132
cc_backend
153-
gcc_backend_impl
154133
cuda_backend_impl
155134
metal_backend_impl
156135
lowered_backend_missing

0 commit comments

Comments
 (0)