add_clang_asan instead of valgrind [WIP] [MOD-10409]

.github/workflows/alpine3.yml

@@ -8,4 +8,4 @@ jobs:
     with:
       container: alpine:3
       pre-checkout-script: apk add bash git
-      run-valgrind: true
+      run-asan: true

.github/workflows/arm.yml

@@ -33,7 +33,7 @@ jobs:
     uses: ./.github/workflows/task-unit-test.yml
     with:
       env: ${{ needs.start-runner.outputs.label }} # run the job on the newly created runner
-      run-valgrind: false # run the job without valgrind
+      run-asan: false # run the job without valgrind
 
   stop-runner:
     name: Stop self-hosted EC2 runner

.github/workflows/flow-temp.yml

@@ -15,23 +15,23 @@ jobs:
   #   uses: ./.github/workflows/task-unit-test.yml
   #   with:
   #     container: ubuntu:jammy
-  #     run-valgrind: true
+  #     run-asan: true
   # alpine3:
   #   uses: ./.github/workflows/task-unit-test.yml
   #   with:
   #     container: alpine:3
   #     pre-checkout-script: apk add bash
-  #     run-valgrind: true
+  #     run-asan: true
   # focal:
   #   uses: ./.github/workflows/task-unit-test.yml
   #   with:
   #     container: ubuntu:focal
-  #     run-valgrind: false
+  #     run-asan: false
   # bionic:
   #   uses: ./.github/workflows/task-unit-test.yml
   #   with:
   #     container: ubuntu:focal
-  #     run-valgrind: false
+  #     run-asan: false
   # bullseye:
   #   uses: ./.github/workflows/task-unit-test.yml
   #   with:

.github/workflows/macos.yml

@@ -7,4 +7,4 @@ jobs:
     uses: ./.github/workflows/task-unit-test.yml
     with:
       env: macos-latest
-      run-valgrind: false
+      run-asan: false

.github/workflows/mariner2.yml

@@ -8,4 +8,4 @@ jobs:
     with:
       container: mcr.microsoft.com/cbl-mariner/base/core:2.0
       pre-checkout-script: tdnf install -y --noplugins --skipsignature tar gzip ca-certificates git
-      run-valgrind: false # TODO: enable valgrind? (requires to install valgrind)
+      run-asan: false # TODO: enable AddressSanitizer? (requires clang)

.github/workflows/task-unit-test.yml

@@ -13,8 +13,8 @@ on:
       pre-checkout-script:
         description: 'Script to run before checkout'
         type: string
-      run-valgrind:
-        description: 'Run valgrind tests'
+      run-asan:
+        description: 'Run AddressSanitizer tests'
         type: boolean
         default: true
 
@@ -67,12 +67,12 @@ jobs:
 
       - name: unit tests
         run: make unit_test
-      - name: valgrind
-        if: ${{ inputs.run-valgrind }}
-        run: make valgrind
-      - name: Archive valgrind tests reports
-        if: ${{ inputs.run-valgrind && failure() }}
+      - name: AddressSanitizer tests
+        if: ${{ inputs.run-asan }}
+        run: make asan
+      - name: Archive AddressSanitizer test reports
+        if: ${{ inputs.run-asan && failure() }}
         uses: actions/upload-artifact@v4
         with:
-          name: valgrind tests reports on ${{ steps.artifact-name.outputs.name }}
+          name: AddressSanitizer test reports on ${{ steps.artifact-name.outputs.name }}
           path: bin/Linux-x86_64-debug/unit_tests/Testing/Temporary/

Makefile

@@ -8,12 +8,12 @@ ifneq ($(filter coverage show-cov upload-cov,$(MAKECMDGOALS)),)
 COV=1
 endif
 
-ifneq ($(VG),)
-VALGRIND=$(VG)
-endif
-
-ifeq ($(VALGRIND),1)
+ifneq ($(ASAN),1)
+# ASAN is not enabled
+else
+# ASAN is enabled - force debug build and set SAN=address
 override DEBUG ?= 1
+override SAN = address
 endif
 
 ifeq ($(COV),1)
@@ -38,7 +38,7 @@ endif
 
 ifeq ($(SAN),memory)
 CMAKE_SAN=-DUSE_MSAN=ON
-override CTEST_ARGS += --exclude-regex BruteForceTest.sanity_rinsert_1280
+#override CTEST_ARGS += --exclude-regex BruteForceTest.sanity_rinsert_1280
 
 else ifeq ($(SAN),address)
 CMAKE_SAN=-DUSE_ASAN=ON
@@ -60,7 +60,7 @@ make build
   DEBUG=1          # build debug variant
   COV=1			   # build for code coverage
   VERBOSE=1        # print detailed build info
-  VG|VALGRIND=1    # build for Valgrind
+  ASAN=1           # build with AddressSanitizer (clang)
   SAN=type         # build with LLVM sanitizer (type=address|memory|leak|thread)
   SLOW=1           # don't run build in parallel (for diagnostics)
   PROFILE=1		   # enable profiling compile flags (and debug symbols) for release type.
@@ -70,9 +70,9 @@ make clean         # remove binary files
 
 make unit_test     # run unit tests
   CTEST_ARGS=args    # extra CTest arguments
-  VG|VALGRIND=1      # run tests with valgrind
+  ASAN=1             # run tests with AddressSanitizer
   FP_64=1			# run tests with 64-bit floating point
-make valgrind      # build for Valgrind and run tests
+make asan          # build with AddressSanitizer and run tests
 make flow_test     # run flow tests (with pytest)
   TEST=file::name    # run specific test
   VERBOSE=1        # print detailed bindings build info
@@ -173,23 +173,18 @@ ifeq ($(VERBOSE),1)
 _CTEST_ARGS += -V
 endif
 
-ifeq ($(VALGRIND),1)
-_CTEST_ARGS += \
-	-T memcheck \
-	--overwrite MemoryCheckCommandOptions="--leak-check=full --fair-sched=yes --error-exitcode=255"
-CMAKE_FLAGS += -DUSE_VALGRIND=ON
-endif
+# AddressSanitizer is handled via SAN=address in cmake/san.cmake
 
 unit_test:
 	$(SHOW)mkdir -p $(BINDIR)
 	$(SHOW)cd $(BINDIR) && cmake $(CMAKE_FLAGS) $(CMAKE_DIR)
 	@make --no-print-directory -C $(BINDIR) $(MAKE_J)
 	$(SHOW)cd $(TESTDIR) && GTEST_COLOR=1 ctest $(_CTEST_ARGS)
 
-valgrind:
-	$(SHOW)$(MAKE) VG=1 unit_test
+asan:
+	$(SHOW)$(MAKE) ASAN=1 unit_test
 
-.PHONY: unit_test valgrind
+.PHONY: unit_test asan
 
 #----------------------------------------------------------------------------------------------
 ifeq ($(VERBOSE),1)

cmake/san.cmake

@@ -2,12 +2,12 @@ option(USE_ASAN "Use AddressSanitizer (clang)" OFF)
 option(USE_MSAN "Use MemorySanitizer (clang)" OFF)
 if (USE_ASAN OR USE_MSAN)
 	# define this before project()
-	find_file(CMAKE_C_COMPILER "clang")
-	find_file(CMAKE_CXX_COMPILER "clang++")
+	find_file(CMAKE_C_COMPILER "clang-18")
+	find_file(CMAKE_CXX_COMPILER "clang++-18")
 	set(CMAKE_LINKER "${CMAKE_C_COMPILER}")
 
 	if (USE_ASAN)
-		set(CLANG_SAN_FLAGS "-fno-omit-frame-pointer -fsanitize=address")
+		set(CLANG_SAN_FLAGS "-fno-omit-frame-pointer -fsanitize=address -fsized-deallocation")
 
 	elseif (USE_MSAN)
 		set(CLANG_SAN_FLAGS "-fno-omit-frame-pointer -fsanitize=memory -fsanitize-memory-track-origins=2")

cmake/svs.cmake

@@ -30,11 +30,11 @@ if(USE_SVS)
         set(SVS_LVQ_SUPPORTED 1)
         # Valgrind does not support AVX512 and Valgrind in running in Debug
         # so disable it if we are in Debug mode
-        string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
-        if(uppercase_CMAKE_BUILD_TYPE STREQUAL "DEBUG")
-            message(STATUS "SVS: Disabling AVX512 support in Debug mode due to Valgrind")
-            set(SVS_NO_AVX512 ON)
-        endif()
+#        string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
+#        if(uppercase_CMAKE_BUILD_TYPE STREQUAL "DEBUG")
+#            message(STATUS "SVS: Disabling AVX512 support in Debug mode due to Valgrind")
+#            set(SVS_NO_AVX512 ON)
+#        endif()
     else()
         set(SVS_LVQ_SUPPORTED 0)
         message(STATUS "SVS LVQ is not supported on this architecture")

tests/unit/CMakeLists.txt

@@ -28,11 +28,8 @@ if(FP64_TESTS)
 	add_definitions(-DFP64_TESTS)
 endif()
 
-option(USE_VALGRIND "Building for Valgrind" OFF)
-if(USE_VALGRIND)
-    add_definitions(-DRUNNING_ON_VALGRIND)
-    message(STATUS "Building with RUNNING_ON_VALGRIND")
-endif()
+# AddressSanitizer is handled via cmake/san.cmake
+# No special definitions needed for ASan
 
 if (CMAKE_HOST_SYSTEM_PROCESSOR MATCHES "(aarch64)|(arm64)|(ARM64)|(armv8)|(armv9)")
 	include(${root}/cmake/aarch64InstructionFlags.cmake)