[PATCH] D21902: [compiler-rt] Fix TLS resource leaking in unittest
Etienne Bergeron via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 30 12:37:05 PDT 2016
etienneb updated this revision to Diff 62401.
etienneb added a comment.
nits
http://reviews.llvm.org/D21902
Files:
lib/sanitizer_common/tests/sanitizer_posix_test.cc
Index: lib/sanitizer_common/tests/sanitizer_posix_test.cc
===================================================================
--- lib/sanitizer_common/tests/sanitizer_posix_test.cc
+++ lib/sanitizer_common/tests/sanitizer_posix_test.cc
@@ -56,6 +56,7 @@
EXPECT_TRUE(destructor_executed);
SpawnThread(GetPthreadDestructorIterations() + 1);
EXPECT_FALSE(destructor_executed);
+ ASSERT_EQ(0, pthread_key_delete(key));
}
TEST(SanitizerCommon, IsAccessibleMemoryRange) {
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21902.62401.patch
Type: text/x-patch
Size: 480 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160630/6075bded/attachment.bin>
More information about the llvm-commits
mailing list