all of the lldb:: types are 64 bit because it needs to be able to hold a pid of any platform (if you are remote debugging for example).  You need to cast to and from ::pid_t inside your host layer<br><div class="gmail_quote">On Fri, Mar 20, 2015 at 6:24 PM Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just for the records, it's uint64_t as defined in lldb_types.h. pid_t is int32_t at least on FreeBSD and Mac OS (didn't check Linux).  Also, clang complains about signed vs unsigned comparison:<br>
<br>
../tools/lldb/source/Plugins/<u></u>Process/FreeBSD/<u></u>ProcessMonitor.cpp:973:49: warning: comparison of integers of different signs: 'lldb::pid_t' (aka 'unsigned long') and 'int' [-Wsign-compare]<br>
<br>
  if ((pid = terminal.Fork(err_str, err_len)) == -1)<br>
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<u></u>~~~~~~~  ^  ~~<br>
<br>
so actually this change make the logic correct in case fork() fails and therefore returns -1 to the parent.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D8491" target="_blank">http://reviews.llvm.org/D8491</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-commits</a><br>
</blockquote></div>