Skip to content

Commit

Permalink
[mycpp/runtime cleanup] Remove Cheney GC
Browse files Browse the repository at this point in the history
We're going to change the object header, and the #ifdef is just
cluttering things up.

We have a good plan for other optimizations, not moving GC!  Mainly:

- SmallStr optimization
- Boxless optimization (tagged pointers)
  • Loading branch information
Andy C committed Aug 10, 2023
1 parent 21edb03 commit 3f68531
Show file tree
Hide file tree
Showing 10 changed files with 7 additions and 646 deletions.
19 changes: 1 addition & 18 deletions mycpp/NINJA_subgraph.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,26 +18,9 @@ def DefineTargets(ru):
deps_base_dir = 'prebuilt/ninja',
template = 'mycpp')

ru.cc_library(
'//mycpp/cheney_heap',
srcs = ['mycpp/cheney_heap.cc'])

# special test with -D
ru.cc_binary(
'mycpp/cheney_heap_test.cc',
deps = ['//mycpp/cheney_heap'],
matrix = [
('cxx', 'asan+cheney'),
('cxx', 'ubsan+cheney'),
('clang', 'ubsan+cheney'),
('clang', 'coverage+cheney'),
],
phony_prefix = 'mycpp-unit')

ru.cc_library(
'//mycpp/runtime',
# TODO: separate into //mycpp/runtime_{marksweep,bumpleak,cheney}
deps = [ '//mycpp/cheney_heap' ],
# Could separate into //mycpp/runtime_{marksweep,bumpleak}
srcs = [
'mycpp/bump_leak_heap.cc',
'mycpp/gc_builtins.cc',
Expand Down
6 changes: 0 additions & 6 deletions mycpp/TEST.sh
Original file line number Diff line number Diff line change
Expand Up @@ -255,12 +255,6 @@ test-runtime() {
run-test-bin $bin
done

for config in cxx-asan+cheney $ubsan_compiler-ubsan+cheney; do
local bin=_bin/$config/mycpp/cheney_heap_test
ninja $bin
run-test-bin $bin
done

# Run other tests with all variants

unit $ubsan_compiler ubsan
Expand Down
283 changes: 0 additions & 283 deletions mycpp/cheney_heap.cc

This file was deleted.

0 comments on commit 3f68531

Please sign in to comment.