Skip to content

Commit 45e974b

Browse files
Clean up headers and CMake deps
1 parent f1fc507 commit 45e974b

File tree

7 files changed

+15
-23
lines changed

7 files changed

+15
-23
lines changed

libc/benchmarks/gpu/CMakeLists.txt

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -38,31 +38,24 @@ add_unittest_framework_library(
3838
SRCS
3939
LibcGpuBenchmark.cpp
4040
LibcGpuBenchmarkMain.cpp
41-
BenchmarkLogger.cpp
4241
HDRS
4342
LibcGpuBenchmark.h
44-
BenchmarkLogger.h
4543
DEPENDS
44+
libc.benchmarks.gpu.timing.timing
4645
libc.hdr.stdint_proxy
47-
libc.src.__support.big_int
48-
libc.src.__support.c_string
4946
libc.src.__support.CPP.string
5047
libc.src.__support.CPP.string_view
5148
libc.src.__support.CPP.type_traits
52-
libc.src.__support.CPP.limits
5349
libc.src.__support.CPP.algorithm
5450
libc.src.__support.CPP.atomic
5551
libc.src.__support.CPP.array
56-
libc.src.__support.fixed_point.fx_rep
57-
libc.src.__support.macros.properties.types
58-
libc.src.__support.OSUtil.osutil
59-
libc.src.__support.uint128
6052
libc.src.__support.FPUtil.fp_bits
6153
libc.src.__support.FPUtil.sqrt
6254
libc.src.__support.fixedvector
63-
libc.src.time.clock
64-
libc.benchmarks.gpu.timing.timing
55+
libc.src.__support.GPU.utils
56+
libc.src.__support.time.gpu.time_utils
6557
libc.src.stdio.printf
58+
libc.src.time.clock
6659
)
6760

6861
add_subdirectory(src)

libc/benchmarks/gpu/LibcGpuBenchmark.cpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
#include "hdr/stdint_proxy.h"
44
#include "src/__support/CPP/algorithm.h"
5-
#include "src/__support/CPP/array.h"
65
#include "src/__support/CPP/atomic.h"
76
#include "src/__support/CPP/string.h"
87
#include "src/__support/FPUtil/FPBits.h"
@@ -12,6 +11,7 @@
1211
#include "src/__support/macros/config.h"
1312
#include "src/__support/time/gpu/time_utils.h"
1413
#include "src/stdio/printf.h"
14+
#include "src/time/clock.h"
1515

1616
namespace LIBC_NAMESPACE_DECL {
1717
namespace benchmarks {
@@ -136,9 +136,11 @@ void print_results(Benchmark *b) {
136136
LIBC_NAMESPACE::printf(
137137
"%-24s |%15.0f |%9.0f |%8llu |%8llu |%11llu |%9u |\n",
138138
b->get_test_name().data(), final_result.cycles,
139-
final_result.standard_deviation, (unsigned long long)final_result.min,
140-
(unsigned long long)final_result.max,
141-
(unsigned long long)final_result.total_iterations, (unsigned)num_threads);
139+
final_result.standard_deviation,
140+
static_cast<unsigned long long>(final_result.min),
141+
static_cast<unsigned long long>(final_result.max),
142+
static_cast<unsigned long long>(final_result.total_iterations),
143+
static_cast<unsigned>(num_threads));
142144
}
143145

144146
void print_header() {

libc/benchmarks/gpu/LibcGpuBenchmark.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
11
#ifndef LLVM_LIBC_BENCHMARKS_LIBC_GPU_BENCHMARK_H
22
#define LLVM_LIBC_BENCHMARKS_LIBC_GPU_BENCHMARK_H
33

4-
#include "benchmarks/gpu/BenchmarkLogger.h"
54
#include "benchmarks/gpu/timing/timing.h"
5+
66
#include "hdr/stdint_proxy.h"
77
#include "src/__support/CPP/algorithm.h"
88
#include "src/__support/CPP/array.h"
9-
#include "src/__support/CPP/limits.h"
109
#include "src/__support/CPP/string_view.h"
1110
#include "src/__support/CPP/type_traits.h"
1211
#include "src/__support/FPUtil/FPBits.h"
1312
#include "src/__support/FPUtil/sqrt.h"
1413
#include "src/__support/macros/config.h"
15-
#include "src/time/clock.h"
1614

1715
namespace LIBC_NAMESPACE_DECL {
1816

libc/benchmarks/gpu/timing/amdgpu/CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@ add_header_library(
44
timing.h
55
DEPENDS
66
libc.hdr.stdint_proxy
7-
libc.src.__support.common
87
libc.src.__support.macros.config
98
libc.src.__support.macros.attributes
109
libc.src.__support.CPP.algorithm
1110
libc.src.__support.CPP.array
11+
libc.src.__support.CPP.atomic
1212
libc.src.__support.CPP.type_traits
13+
libc.src.__support.GPU.utils
1314
)

libc/benchmarks/gpu/timing/amdgpu/timing.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
#include "src/__support/CPP/atomic.h"
1616
#include "src/__support/CPP/type_traits.h"
1717
#include "src/__support/GPU/utils.h"
18-
#include "src/__support/common.h"
1918
#include "src/__support/macros/attributes.h"
2019
#include "src/__support/macros/config.h"
2120

libc/benchmarks/gpu/timing/nvptx/CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@ add_header_library(
44
timing.h
55
DEPENDS
66
libc.hdr.stdint_proxy
7-
libc.src.__support.common
87
libc.src.__support.macros.config
98
libc.src.__support.macros.attributes
109
libc.src.__support.CPP.algorithm
1110
libc.src.__support.CPP.array
11+
libc.src.__support.CPP.atomic
1212
libc.src.__support.CPP.type_traits
13+
libc.src.__support.GPU.utils
1314
)

libc/benchmarks/gpu/timing/nvptx/timing.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,7 @@
1313
#include "src/__support/CPP/algorithm.h"
1414
#include "src/__support/CPP/array.h"
1515
#include "src/__support/CPP/atomic.h"
16-
#include "src/__support/CPP/type_traits.h"
1716
#include "src/__support/GPU/utils.h"
18-
#include "src/__support/common.h"
1917
#include "src/__support/macros/attributes.h"
2018
#include "src/__support/macros/config.h"
2119

0 commit comments

Comments
 (0)