This is an archive of the discontinued LLVM Phabricator instance.

Make tsan_analyze critical for the bots again.
ClosedPublic

Authored by krasin on Sep 11 2016, 12:38 PM.

Reviewers

Commits

Summary

Since the tsan_analyze failure is fixed, it's time to upgrade this
check to the critical one. This reverts r280761.

krasin retitled this revision from to Make tsan_analyze critical for the bots again..

krasin updated this object.

krasin added a reviewer: kcc.

This revision was automatically updated to reflect the committed changes.

Path

Size

zorg/

buildbot/

builders/

sanitizers/

2 lines

	Show First 20 Lines • Show All 58 Lines • ▼ Show 20 Lines
	build_tsan "${TSAN_FULL_DEBUG_BUILD_DIR}" "-DCOMPILER_RT_DEBUG=ON -DCOMPILER_RT_TSAN_DEBUG_OUTPUT=ON -DLLVM_INCLUDE_TESTS=OFF" gcc g++			build_tsan "${TSAN_FULL_DEBUG_BUILD_DIR}" "-DCOMPILER_RT_DEBUG=ON -DCOMPILER_RT_TSAN_DEBUG_OUTPUT=ON -DLLVM_INCLUDE_TESTS=OFF" gcc g++

	echo @@@BUILD_STEP build release tsan with clang@@@			echo @@@BUILD_STEP build release tsan with clang@@@
	build_tsan "${TSAN_RELEASE_BUILD_DIR}" "-DCOMPILER_RT_DEBUG=OFF" "$ROOT/$TSAN_DEBUG_BUILD_DIR/bin/clang" "$ROOT/$TSAN_DEBUG_BUILD_DIR/bin/clang++"			build_tsan "${TSAN_RELEASE_BUILD_DIR}" "-DCOMPILER_RT_DEBUG=OFF" "$ROOT/$TSAN_DEBUG_BUILD_DIR/bin/clang" "$ROOT/$TSAN_DEBUG_BUILD_DIR/bin/clang++"

	echo @@@BUILD_STEP tsan analyze@@@			echo @@@BUILD_STEP tsan analyze@@@
	BIN=$(mktemp -t tsan_exe.XXXXXXXX)			BIN=$(mktemp -t tsan_exe.XXXXXXXX)
	echo "int main() {return 0;}" \| $TSAN_RELEASE_BUILD_DIR/bin/clang -x c++ - -fsanitize=thread -O2 -o ${BIN}			echo "int main() {return 0;}" \| $TSAN_RELEASE_BUILD_DIR/bin/clang -x c++ - -fsanitize=thread -O2 -o ${BIN}
	$LLVM_CHECKOUT/projects/compiler-rt/lib/tsan/check_analyze.sh ${BIN} \|\| echo @@@STEP_WARNINGS@@@			$LLVM_CHECKOUT/projects/compiler-rt/lib/tsan/check_analyze.sh ${BIN} \|\| echo @@@STEP_FAILURE@@@