[Lldb-commits] [PATCH] D44056: [lldb] Fix "code unreachable" warning in HostThreadPosix::Cancel
Adrian Prantl via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Mar 2 16:55:02 PST 2018
aprantl added inline comments.
================
Comment at: source/Host/posix/HostThreadPosix.cpp:44
if (IsJoinable()) {
#ifndef __ANDROID__
#ifndef __FreeBSD__
----------------
What about:
```
#ifdef __ANDROID__
error.SetErrorString("HostThreadPosix::Cancel() not supported on Android");
#else
#ifdef __FreeBSD__
int err = ::pthread_cancel(m_thread);
error.SetError(err, eErrorTypePOSIX);
#else
llvm_unreachable("someone is calling HostThread::Cancel()");
#endif
#endif
}
return error;
}
```
https://reviews.llvm.org/D44056
More information about the lldb-commits
mailing list