Skip to content

Commit

Permalink
[build fix] Finish removing Cheney GC
Browse files Browse the repository at this point in the history
  • Loading branch information
Andy C committed Aug 10, 2023
1 parent 3f68531 commit ed3f1ff
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 11 deletions.
4 changes: 1 addition & 3 deletions build/native.sh
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,9 @@ soil-run() {

./NINJA-config.sh

# Keep the Cheney build compiling
ninja _bin/cxx-dbg/oils-for-unix \
_bin/cxx-asan/oils-for-unix \
_bin/cxx-opt/oils-for-unix.stripped \
_bin/cxx-opt+cheney/oils-for-unix
_bin/cxx-opt/oils-for-unix.stripped
}

oil-slice-demo() {
Expand Down
6 changes: 1 addition & 5 deletions build/ninja-rules-cpp.sh
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ setglobal_compile_flags() {
fi

case $variant in
*+bumpleak|*+bumproot|*+cheney)
*+bumpleak|*+bumproot)
;;
*)
flags="$flags -D MARK_SWEEP"
Expand Down Expand Up @@ -148,10 +148,6 @@ setglobal_compile_flags() {
flags="$flags -D GC_ALWAYS"
;;

*+cheney)
flags="$flags -D CHENEY_GC"
;;

*+tcmalloc)
flags="$flags -D TCMALLOC"
;;
Expand Down
4 changes: 2 additions & 2 deletions metrics/source-code.sh
Original file line number Diff line number Diff line change
Expand Up @@ -162,10 +162,10 @@ cpp-counts() {
"$@"

# Remove code that isn't "in production"
ls mycpp/*.{cc,h} | egrep -v '_test.cc|cheney_heap|bump_leak_heap' \
ls mycpp/*.{cc,h} | egrep -v '_test.cc|bump_leak_heap' \
| $count \
'Garbage-Collected Runtime' \
'Uses a simple Cheney / semi-space collector.' \
'Uses a fork-friendly Mark-Sweep collector.' \
"$@"

ls mycpp/*_test.cc cpp/*_test.cc | $count \
Expand Down
1 change: 0 additions & 1 deletion test/coverage.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ html-report() {
--ignore-filename-regex '_gen/' \
--ignore-filename-regex 'mycpp/demo' \
--ignore-filename-regex 'mycpp/examples' \
--ignore-filename-regex 'mycpp/cheney' \
--ignore-filename-regex 'prebuilt/' \
)

Expand Down

0 comments on commit ed3f1ff

Please sign in to comment.