<div dir="ltr"><div>Thanks for the heads up.  I do run with asserts, so I'm not sure why I didn't see this particular failure.  In any case i'll go back and look through my changes for any other occurrences of this.</div><div><br></div><div>Most of the time args were being checked before being used.  Like the code would go:</div><div><br></div><div>const char *arg = args.GetArgumentAtIndex(i);</div><div>if (!arg || arg[0] == '\0')</div><div>  continue;</div><div><br></div><div>// Use it.</div><div><br></div><div>I shouldn't have messed up any of those because I converted them all to</div><div><br></div><div>if (arg.empty())</div><div>  continue;</div><div><br></div><div>So hopefully the one you found is the only one.</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Oct 6, 2016 at 12:16 PM Jim Ingham <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I fixed a bug in CommandObject::Execute that cause test-suite crashes when running with an Assert build of llvm (r283479.)<br class="gmail_msg">
<br class="gmail_msg">
The test was doing:<br class="gmail_msg">
<br class="gmail_msg">
break modify -c ''<br class="gmail_msg">
<br class="gmail_msg">
which is the way you unset a condition on a breakpoint.<br class="gmail_msg">
<br class="gmail_msg">
So the StringRef for that option value was empty, and it appears that you aren't supposed to call "front" on an empty StringRef.  But empty option values are used in a bunch of places in lldb.<br class="gmail_msg">
<br class="gmail_msg">
Zachary, you might want to go scrub through the Args changes (and anywhere else you might be parsing an empty StringRef) to make sure there aren't other errors of this sort.<br class="gmail_msg">
<br class="gmail_msg">
And if you aren't running your prep testsuite runs with an Assert build of llvm, it's probably a good idea to do so.<br class="gmail_msg">
<br class="gmail_msg">
Jim<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>