build(cmake): remove unnecessary lines

c0nstexpr · c0nstexpr · commit 2fee9bc7ed3e · 2024-02-20T10:04:12.000+08:00
diff --git a/.github/actions/apt-fetch-clang/action.yml b/.github/actions/apt-fetch-clang/action.yml
@@ -2,8 +2,7 @@ name: Fetch Clang toolchain on Linux
 
 inputs:
   llvm-ver:
-    required: false
-    default: 17
+    required: true
 runs:
   using: composite
 
diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -25,6 +25,8 @@ jobs:
 
       - name: Fetch Clang
         uses: ./.github/actions/apt-fetch-clang
+        with:
+          llvm-ver: 17
 
       - name: Get cmake option
         id: cmake-option
@@ -76,7 +78,7 @@ jobs:
 
       - uses: ilammy/msvc-dev-cmd@v1
         with:
-          toolset: 14.38
+          toolset: 14.39
 
       - name: CMake build and test (Debug)
         uses: ./.github/actions/cmake-build-and-test
diff --git a/cmake/Utils.cmake b/cmake/Utils.cmake
@@ -13,16 +13,6 @@ if("${CMAKE_CXX_COMPILER_ID}" MATCHES "MSVC")
   add_compile_options(/utf-8 /diagnostics:caret)
 endif()
 
-if("${CMAKE_BUILD_TYPE}" MATCHES "Debug")
-  message(STATUS "Debug mode.\n")
-else()
-  message(
-    STATUS
-      "Current config is ${CMAKE_BUILD_TYPE}, not Debug mode, add NDEBUG definition\n"
-  )
-  add_compile_definitions("NDEBUG")
-endif()
-
 include(ProcessorCount)
 
 ProcessorCount(PROCESSOR_COUNT)
