Todd Lipcon has submitted this change and it was merged. Change subject: Remove TSAN suppressions for protobuf ......................................................................
Remove TSAN suppressions for protobuf We now build protobuf with TSAN instrumentation enabled. So, we shouldn't need to suppress anything from the library. This required adding -DTHREAD_SANITIZER to the instrumented protobuf build so that the TSAN instrumentation got built in. Change-Id: I0b4a2066dfc4b4e83c5c0a35bb8e5aa8f42d9147 Reviewed-on: http://gerrit.cloudera.org:8080/1990 Tested-by: Todd Lipcon <[email protected]> Reviewed-by: David Ribeiro Alves <[email protected]> --- M build-support/tsan-suppressions.txt M thirdparty/build-definitions.sh M thirdparty/build-thirdparty.sh 3 files changed, 4 insertions(+), 6 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Todd Lipcon: Verified -- To view, visit http://gerrit.cloudera.org:8080/1990 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b4a2066dfc4b4e83c5c0a35bb8e5aa8f42d9147 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
