Skip to content

Commit

Permalink
remove byte code size stuff
Browse files Browse the repository at this point in the history
  • Loading branch information
Andy C committed Aug 25, 2024
1 parent 1389d21 commit 31f9611
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 38 deletions.
23 changes: 0 additions & 23 deletions benchmarks/ovm-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -116,12 +116,6 @@ measure-sizes() {
sizes-tsv $BASE_DIR/bin/*/{oils-for-unix,oils-for-unix.stripped} \
> ${prefix}.native-sizes.tsv

sizes-tsv $TAR_DIR/oil-$OIL_VERSION/_build/oil/bytecode-opy.zip \
> ${prefix}.bytecode-size.tsv

sizes-tsv $BASE_DIR/bin/*/oil.* \
> ${prefix}.bin-sizes.tsv

sizes-tsv $BASE_DIR/bin/*/*sh \
> ${prefix}.other-shell-sizes.tsv

Expand Down Expand Up @@ -384,11 +378,6 @@ stage1() {
b=($raw_dir/$MACHINE2.*.times.tsv)
tsv-concat ${a[-1]} ${b[-1]} > $x

x=$out/bytecode-size.tsv
a=($raw_dir/$MACHINE1.*.bytecode-size.tsv)
b=($raw_dir/$MACHINE2.*.bytecode-size.tsv)
tsv-concat ${a[-1]} ${b[-1]} > $x

x=$out/bin-sizes.tsv
a=($raw_dir/$MACHINE1.*.bin-sizes.tsv)
b=($raw_dir/$MACHINE2.*.bin-sizes.tsv)
Expand Down Expand Up @@ -446,18 +435,6 @@ EOF
EOF
tsv2html --css-class-pattern 'special ^gcc' $in_dir/native-sizes.tsv

cmark << 'EOF'
### OVM Binary Size
The oil binary has two portions:
- Architecture-independent `bytecode.zip`
- Architecture- and compiler- dependent native code (`_build/oil/ovm*`)
EOF
# Highlight the "default" production build
tsv2html --css-class-pattern 'special /gcc/oil.ovm$' $in_dir/sizes.tsv

cmark << 'EOF'
### Host and Compiler Details
Expand Down
15 changes: 0 additions & 15 deletions benchmarks/report.R
Original file line number Diff line number Diff line change
Expand Up @@ -674,7 +674,6 @@ WriteOvmBuildDetails = function(distinct_hosts, distinct_compilers, out_dir) {

OvmBuildReport = function(in_dir, out_dir) {
times = readTsv(file.path(in_dir, 'times.tsv'))
bytecode_size = readTsv(file.path(in_dir, 'bytecode-size.tsv'))
bin_sizes = readTsv(file.path(in_dir, 'bin-sizes.tsv'))
native_sizes = readTsv(file.path(in_dir, 'native-sizes.tsv'))
raw_data = readTsv(file.path(in_dir, 'raw-data.tsv'))
Expand Down Expand Up @@ -716,18 +715,6 @@ OvmBuildReport = function(in_dir, out_dir) {

#print(times)

bytecode_size %>%
rename(bytecode_size = num_bytes) %>%
select(-c(path)) ->
bytecode_size

bin_sizes %>%
# reorder
select(c(host_label, path, num_bytes)) %>%
left_join(bytecode_size, by = c('host_label')) %>%
mutate(native_code_size = num_bytes - bytecode_size) ->
sizes

# paths look like _tmp/ovm-build/bin/clang/oils_cpp.stripped
native_sizes %>%
select(c(host_label, path, num_bytes)) %>%
Expand All @@ -742,8 +729,6 @@ OvmBuildReport = function(in_dir, out_dir) {

# NOTE: These don't have the host and compiler.
writeTsv(times, file.path(out_dir, 'times'))
writeTsv(bytecode_size, file.path(out_dir, 'bytecode-size'))
writeTsv(sizes, file.path(out_dir, 'sizes'))
writeTsv(native_sizes, file.path(out_dir, 'native-sizes'))

# TODO: I want a size report too
Expand Down

0 comments on commit 31f9611

Please sign in to comment.