Skip to content

Commit b4cd893

Browse files
committed
cleanup
1 parent 5c98906 commit b4cd893

File tree

3 files changed

+4
-30
lines changed

3 files changed

+4
-30
lines changed

CMakeLists.txt

Lines changed: 1 addition & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -275,33 +275,7 @@ include(add_nlohmann_json)
275275
add_nlohmann_json()
276276
add_json_schema_validator()
277277

278-
# StableCoder cmake-scripts
279-
# TODO: ask fabian if not release or just equal debug is better
280-
281-
# TODO: add install lcov to bootstrap or something
282-
# Do NOT do: sudo apt-get install -y lcov
283-
# Apparently this version is insanely old and we need a newer version/the newest version
284-
# Installing lcov via the github page + make install has four billion dependencies, so
285-
# we need to install those too:
286-
# These perl packages include:
287-
#
288-
# - Capture::Tiny
289-
# - DateTime
290-
# - Devel::Cover
291-
# - Digest::MD5
292-
# - File::Spec
293-
# - at least one flavor of JSON module.
294-
# In order of performance/preference:
295-
# - JSON::XS
296-
# - Cpanel::JSON::XS
297-
# - JSON::PP
298-
# - JSON
299-
# - Memory::Process
300-
# - Module::Load::Conditional
301-
# - Scalar::Util
302-
# - Time::HiRes
303-
# - TimeDate
304-
278+
# Coverage
305279
if ("${CMAKE_BUILD_TYPE}" STREQUAL "Debug" AND CODE_COVERAGE)
306280
find_program(LLVM_COV_PATH llvm-cov-19)
307281
message(STATUS "Found llvm-cov ${LLVM_COV_PATH}")

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ RUN --mount=type=bind,source=.,target=/usr/src/phasar,rw \
1818
cmake -S . -B cmake-build/Release \
1919
-DCMAKE_BUILD_TYPE=Release \
2020
-DPHASAR_TARGET_ARCH="" \
21-
-DPHASAR_ENABLE_SANITIZERS=OFF \
21+
-DPHASAR_ENABLE_SANITIZERS=ON \
2222
-DBUILD_PHASAR_CLANG=ON \
2323
-DPHASAR_USE_Z3=ON \
2424
-DPHASAR_BUILD_UNITTESTS=$RUN_TESTS \

cmake/phasar_macros.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ function(add_phasar_unittest test_name)
3636
set_tests_properties("${test}" PROPERTIES LABELS "all")
3737
set(CTEST_OUTPUT_ON_FAILURE ON)
3838

39-
if ("${CMAKE_BUILD_TYPE}" STREQUAL "Debug" AND CODE_COVERAGE AND NOT PHASAR_DEBUG_LIBDEPS)
39+
if ("${CMAKE_BUILD_TYPE}" STREQUAL "Debug" AND CODE_COVERAGE)
4040
target_code_coverage(${test} AUTO ALL)
4141
endif()
4242
endfunction()
@@ -356,7 +356,7 @@ function(add_phasar_library name)
356356

357357
set_property(GLOBAL APPEND PROPERTY LLVM_EXPORTS ${name})
358358

359-
if ("${CMAKE_BUILD_TYPE}" STREQUAL "Debug" AND CODE_COVERAGE AND NOT PHASAR_DEBUG_LIBDEPS)
359+
if ("${CMAKE_BUILD_TYPE}" STREQUAL "Debug" AND CODE_COVERAGE)
360360
target_code_coverage(${name} AUTO ALL)
361361
endif()
362362
endfunction(add_phasar_library)

0 commit comments

Comments
 (0)