You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2018/08/28 02:45:35 UTC
[kudu-CR] [build] add -D{LEAK,UNDEFINED} SANITIZER
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11340
to look at the new patch set (#2).
Change subject: [build] add -D{LEAK,UNDEFINED}_SANITIZER
......................................................................
[build] add -D{LEAK,UNDEFINED}_SANITIZER
To enable __lsan_default_suppressions() defined in
src/kudu/util/sanitizer_options.cc, it's necessary to have
LEAK_SANITIZER macro enabled during the compilation.
Also, defined UNDEFINED_SANITIZER macro to compile-in
__ubsan_default_options() as well.
This patch fixes negotiation-test failures in ASAN builds.
This is a follow-up for a8d088e11b8b901a91e4820dcfdca9537a1e801a.
Change-Id: Ie8caeaa080b93f67768d3d2251c7a366390a48f9
---
M CMakeLists.txt
1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/40/11340/2
--
To view, visit http://gerrit.cloudera.org:8080/11340
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie8caeaa080b93f67768d3d2251c7a366390a48f9
Gerrit-Change-Number: 11340
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins