summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRichard Trieu <rtrieu@google.com>2014-02-26 03:12:51 +0000
committerRichard Trieu <rtrieu@google.com>2014-02-26 03:12:51 +0000
commita170f081092b7d1ad966e5cc57c0b18a54778642 (patch)
tree694ab41a86421cd46daca9ffa3e2f1dfdb42820a
parentea08c7090b7eded5351ca8d63e6312bb28074dd2 (diff)
downloadbcm5719-llvm-a170f081092b7d1ad966e5cc57c0b18a54778642.tar.gz
bcm5719-llvm-a170f081092b7d1ad966e5cc57c0b18a54778642.zip
Remove 'tctx->name' from a logical statement since it is a pointer and always
is converted to a true value. Detected by Clang's improved -Wbool-conversion llvm-svn: 202223
-rw-r--r--compiler-rt/lib/sanitizer_common/tests/sanitizer_thread_registry_test.cc2
1 files changed, 1 insertions, 1 deletions
diff --git a/compiler-rt/lib/sanitizer_common/tests/sanitizer_thread_registry_test.cc b/compiler-rt/lib/sanitizer_common/tests/sanitizer_thread_registry_test.cc
index ddc8dba5d3e..69b6e13e721 100644
--- a/compiler-rt/lib/sanitizer_common/tests/sanitizer_thread_registry_test.cc
+++ b/compiler-rt/lib/sanitizer_common/tests/sanitizer_thread_registry_test.cc
@@ -48,7 +48,7 @@ static uptr get_uid(u32 tid) {
static bool HasName(ThreadContextBase *tctx, void *arg) {
char *name = (char*)arg;
- return (tctx->name && 0 == internal_strcmp(tctx->name, name));
+ return (0 == internal_strcmp(tctx->name, name));
}
static bool HasUid(ThreadContextBase *tctx, void *arg) {
OpenPOWER on IntegriCloud