minor fixes

Author: jameshu15869

libc/benchmarks/gpu/LibcGpuBenchmark.cpp

@@ -136,9 +136,9 @@ void print_header() {
 void Benchmark::run_benchmarks() {
   uint64_t id = gpu::get_thread_id();
 
-  if (id == 0) {
+  if (id == 0)
     print_header();
-  }
+
   gpu::sync_threads();
 
   for (Benchmark *b : benchmarks) {

libc/benchmarks/gpu/src/math/CMakeLists.txt

@@ -27,6 +27,7 @@ add_benchmark(
   COMPILE_OPTIONS
     ${nvptx_math_found}
     ${nvptx_bitcode_link_flags}
+    -save-temps
   LOADER_ARGS
     --threads 64
 )

libc/benchmarks/gpu/src/math/sin_benchmark.cpp

@@ -17,8 +17,6 @@ uint64_t get_bits(double x) {
   return LIBC_NAMESPACE::cpp::bit_cast<uint64_t>(x);
 }
 
-constexpr int RANDOM_INPUT_SIZE = 256;
-
 // BENCHMARK() expects a function that with no parameters that returns a
 // uint64_t representing the latency. Defining each benchmark using macro that
 // expands to a lambda to allow us to switch the implementation of `sin()` to

libc/cmake/modules/LLVMLibCCompileOptionRules.cmake

@@ -230,7 +230,7 @@ endfunction()
 function(_get_hermetic_test_compile_options output_var flags)
   _get_compile_options_from_flags(compile_flags ${flags})
   list(APPEND compile_options ${LIBC_COMPILE_OPTIONS_DEFAULT} ${compile_flags}
-       ${flags} -fpie -ffreestanding -fno-exceptions -fno-rtti)
+       ${flags} -fpie -ffreestanding -fno-exceptions -fno-rtti -save-temps)
 
   # The GPU build requires overriding the default CMake triple and architecture.
   if(LIBC_TARGET_ARCHITECTURE_IS_AMDGPU)