[scudo] Do not compile timing.cpp if LLVM_LIBC_INCLUDE_SCUDO=on

Temporary hack until LLVM libc supports inttypes.h print format macros

timing.h uses the PRId64 macro which is not included in llvm libc yet

Bug: https://github.com/llvm/llvm-project/issues/63317

Reviewed By: michaelrj, thesamesam, Chia-hungDuan

Differential Revision: https://reviews.llvm.org/D152979
This commit is contained in:
Alfred Persson Forsberg 2023-06-16 00:08:57 +01:00 committed by Sam James
parent 0cbbfb8c2e
commit 63eb7c4e66
2 changed files with 13 additions and 0 deletions

View File

@ -114,6 +114,13 @@ set(SCUDO_SOURCES
timing.cpp
)
# Temporary hack until LLVM libc supports inttypes.h print format macros
# See: https://github.com/llvm/llvm-project/issues/63317#issuecomment-1591906241
if(LLVM_LIBC_INCLUDE_SCUDO)
list(REMOVE_ITEM SCUDO_HEADERS timing.h)
list(REMOVE_ITEM SCUDO_SOURCES timing.cpp)
endif()
# Enable the necessary instruction set for scudo_crc32.cpp, if available.
# Newer compiler versions use -mcrc32 rather than -msse4.2.
if (COMPILER_RT_HAS_MCRC32_FLAG)

View File

@ -111,6 +111,12 @@ set(SCUDO_UNIT_TEST_SOURCES
scudo_unit_test_main.cpp
)
# Temporary hack until LLVM libc supports inttypes.h print format macros
# See: https://github.com/llvm/llvm-project/issues/63317#issuecomment-1591906241
if(LLVM_LIBC_INCLUDE_SCUDO)
list(REMOVE_ITEM SCUDO_UNIT_TEST_SOURCES timing_test.cpp)
endif()
add_scudo_unittest(ScudoUnitTest
SOURCES ${SCUDO_UNIT_TEST_SOURCES})