Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

build: add sanitizer support #359

Merged
merged 15 commits into from
Dec 18, 2019
Merged

Conversation

foreverneverer
Copy link
Contributor

@foreverneverer foreverneverer commented Dec 17, 2019

This pr add --sanitizer(sanitizers ) options that contain "address, leak, thread, undefined". You can run :
./run.sh test --skip_thirdparty --check --disable_gperf --sanitizer address
check potential problems of your code.

run.sh Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
run.sh Outdated Show resolved Hide resolved
run.sh Outdated Show resolved Hide resolved
run.sh Outdated Show resolved Hide resolved
run.sh Outdated Show resolved Hide resolved
bin/dsn.cmake Outdated Show resolved Hide resolved
scripts/linux/build.sh Outdated Show resolved Hide resolved
bin/dsn.cmake Outdated Show resolved Hide resolved
@vagetablechicken
Copy link
Member

pr名改一下吧,for cmake这个说法有点奇怪。就是给build加入了sanitizer,或者叫sanitizer support

@foreverneverer foreverneverer changed the title build: add sanitizer options for cmake build: add sanitizer support Dec 18, 2019
@qinzuoyan qinzuoyan merged commit 55634c4 into XiaoMi:master Dec 18, 2019
bin/dsn.cmake Show resolved Hide resolved
run.sh Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants