Skip to content

Commit 88b5f6a

Browse files
authored
samples/wasm-c-api: remove unused valgrind detection (#4249)
- it's unused - valgrind is basically a linux-only software. it isn't a good idea to make it a hard requirement. if we want to use valgrind, it's better to introduce a separate option to control it.
1 parent ac2fe55 commit 88b5f6a

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

samples/wasm-c-api/CMakeLists.txt

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -205,12 +205,3 @@ foreach(EX ${EXAMPLES})
205205
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
206206
)
207207
endforeach()
208-
209-
if (CMAKE_BUILD_TYPE STREQUAL "Debug")
210-
find_program(VALGRIND
211-
valgrind
212-
REQUIRED
213-
)
214-
215-
# run `ctest -T memcheck -V --test-dir build`
216-
endif()

0 commit comments

Comments
 (0)