[scudo] Temporarily disable the build of Timer.
authorChia-hung Duan <chiahungduan@google.com>
Tue, 4 Apr 2023 00:58:40 +0000 (00:58 +0000)
committerChia-hung Duan <chiahungduan@google.com>
Tue, 4 Apr 2023 01:04:24 +0000 (01:04 +0000)
The placeholder was %llu while merged but it may cause problem on some
platforms that enable -Wformat. It was incorrect changed to %lu which
cause the test failure because string_utils doesn't support %lu. Will
bring this back after we fix the placeholder.

Differential Revision: https://reviews.llvm.org/D147490

compiler-rt/lib/scudo/standalone/CMakeLists.txt
compiler-rt/lib/scudo/standalone/tests/CMakeLists.txt

index 6fcd4de..86d4a47 100644 (file)
@@ -85,7 +85,9 @@ set(SCUDO_HEADERS
   stack_depot.h
   stats.h
   string_utils.h
-  timing.h
+  # TODO(chiahungduan): Temporarily disable the timer because of the print
+  # of u64 has incorrect placeholder.
+  # timing.h
   tsd_exclusive.h
   tsd_shared.h
   tsd.h
@@ -108,7 +110,9 @@ set(SCUDO_SOURCES
   report.cpp
   rss_limit_checker.cpp
   string_utils.cpp
-  timing.cpp
+  # TODO(chiahungduan): Temporarily disable the timer because of the print
+  # of u64 has incorrect placeholder.
+  # timing.cpp
   )
 
 # Enable the necessary instruction set for scudo_crc32.cpp, if available.
index 335e4b7..3bb707d 100644 (file)
@@ -105,7 +105,9 @@ set(SCUDO_UNIT_TEST_SOURCES
   size_class_map_test.cpp
   stats_test.cpp
   strings_test.cpp
-  timing_test.cpp
+  # TODO(chiahungduan): Temporarily disable the timer test because of the print
+  # of u64 has incorrect placeholder.
+  # timing_test.cpp
   tsd_test.cpp
   vector_test.cpp
   scudo_unit_test_main.cpp