<div dir="ltr">Hmm, I'mm seeing some warnings on the debugserver Xcode build that make it look like it may have just switched from a 32-bit build to a 64-bit build on the "lldb-tool | My Mac 64-bit" scheme.<div>
<br></div><div>For example, I'm seeing what I think are new warnings where system calls are returning 64 bits but they're placed in uint32_t and the like, and warning about losing integer precision.</div><div><br>
</div><div>I might have just been ignoring those before, though, and not realized it. Any thoughts, Greg? I will revert the change (or tweak if that's simple) based on whatever you need me to do here.</div><div><br>
</div><div>FWIW the debugserver gdb-remote protocol tests I have in TOT are all working fine.</div><div><br></div><div>Sample warnings follow (might be that -Wshorten-64-to-32 was added as part of the settings upgrade - so these may have always been there, just not noisy).</div>
<div><br></div><div>=====</div><div>
<p class="">In file included from /Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:31:</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/DNBDataRef.h:61:47: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'const offset_t' (aka 'const unsigned int') [-Wshorten-64-to-32]</p>
<p class=""> const offset_t size = GetSize();</p>
<p class=""> ~~~~ ^~~~~~~~~</p>
<p class="">In file included from /Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:34:</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/DNBTimer.h:89:29: warning: implicit conversion loses integer precision: 'const __darwin_time_t' (aka 'const long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32]</p>
<p class=""> sec = m_timeval.tv_sec;</p>
<p class=""> ~ ~~~~~~~~~~^~~~~~</p>
<p class="">In file included from /Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:35:</p>
<p class="">In file included from /Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.h:21:</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachException.h:80:24: warning: implicit conversion loses integer precision: 'const value_type' (aka 'const long long') to 'int' [-Wshorten-64-to-32]</p>
<p class=""> return exc_data[1];</p>
<p class=""> ~~~~~~ ^~~~~~~~~~~</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:1489:34: warning: implicit conversion loses integer precision: 'ssize_t' (aka 'long') to 'int' [-Wshorten-64-to-32]</p>
<p class=""> bytes_read = ::read (stdout_fd, s, sizeof(s)-1);</p>
<p class=""> ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:1513:34: warning: implicit conversion loses integer precision: 'ssize_t' (aka 'long') to 'int' [-Wshorten-64-to-32]</p>
<p class=""> bytes_read = ::read (stderr_fd, s, sizeof(s)-1);</p>
<p class=""> ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:1603:16: warning: implicit conversion loses integer precision: 'uint64_t' (aka 'unsigned long long') to 'useconds_t' (aka 'unsigned int') [-Wshorten-64-to-32]</p>
<p class=""> usleep(proc->ProfileInterval());</p>
<p class=""> ~~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:2266:24: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'u_int' (aka 'unsigned int') [-Wshorten-64-to-32]</p>
<p class=""> if (::sysctl (mib, len, &cpu, &cpu_len, 0, 0))</p>
<p class=""> ~~ ^~~</p>
<p class="">/Users/tfiala/lldb/svn/lldb/tools/debugserver/source/MacOSX/MachProcess.mm:42:20: warning: unused function 'CopyBundleIDForPath' [-Wunused-function]</p>
<p class="">static CFStringRef CopyBundleIDForPath (const char *app_bundle_path, DNBError &err_str);</p>
<p class=""> ^</p><p class=""><br></p><p class=""><br></p></div><div><br></div><div>-Todd</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 19, 2014 at 10:06 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Done:<div><br><div><div>tfiala-macbookpro:lldb tfiala$ svn commit</div><div>Sending lldb.xcodeproj/project.pbxproj</div>
<div>Sending lldb.xcodeproj/xcshareddata/xcschemes/LLDB.xcscheme</div><div>
Sending lldb.xcodeproj/xcshareddata/xcschemes/Run Testsuite.xcscheme</div><div>Sending lldb.xcodeproj/xcshareddata/xcschemes/darwin-debug.xcscheme</div><div>Sending lldb.xcodeproj/xcshareddata/xcschemes/launcherRootXPCService.xcscheme</div>
<div>Sending lldb.xcodeproj/xcshareddata/xcschemes/launcherXPCService.xcscheme</div><div>Sending lldb.xcodeproj/xcshareddata/xcschemes/lldb-tool.xcscheme</div><div>Sending tools/debugserver/debugserver.xcodeproj/project.pbxproj</div>
<div>Sending tools/debugserver/debugserver.xcodeproj/xcshareddata/xcschemes/debugserver.xcscheme</div><div>Transmitting file data .........</div><div>Committed revision 209142.</div></div></div></div><div class="gmail_extra">
<div><div class="h5">
<br><br><div class="gmail_quote">On Mon, May 19, 2014 at 9:50 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Okay, will do, thanks!</div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On Mon, May 19, 2014 at 9:38 AM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Please, feel free to update the settings and check them in.<br>
<div><div><br>
> On May 18, 2014, at 8:31 PM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>> wrote:<br>
><br>
> Hey guys,<br>
><br>
> I consistently get a couple warnings on lldb.xcodeproj and debugserver.xcodeproj about "Update to recommended settings". Is there any reason why we couldn't run that upgrade and check it in? (Some kind of backwards compatibility we need to maintain?)\<br>
><br>
> Thanks!<br>
><br>
> --<br>
> -Todd<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div dir="ltr">-Todd</div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr">-Todd</div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">-Todd</div>
</div>