You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, address sanitization is not enabled in the Build workflow because it causes the argtable3 tests to fail. Once we either remove arguable or fix it, we can enable ASAN in testing.
The text was updated successfully, but these errors were encountered:
We need to check if we are using Clang in the CMakeLists. Just run for the Spatter tests - we think that argtable tests are not included since #117 was merged into the mainline.
// Update to check for CLANG
IF(CMAKE_BUILD_TYPE STREQUAL "Debug")
# Use ASAN when debugging
SET( CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address")
ENDIF(CMAKE_BUILD_TYPE STREQUAL "Debug")
Related to #103 which reports an error with parse_p.
We may want to update the CMakeLists to add the following:
Use for debugging. TODO: Set this automatically
IF(CMAKE_BUILD_TYPE STREQUAL "Debug")
IF ("${CMAKE_C_COMPILER_ID}" STREQUAL "Clang"))
# Use ASAN when debugging
SET( CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address")
ENDIF ()
ENDIF ()
Right now, address sanitization is not enabled in the Build workflow because it causes the argtable3 tests to fail. Once we either remove arguable or fix it, we can enable ASAN in testing.
The text was updated successfully, but these errors were encountered: