<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">I never <b>ran LLDB tests</b>, not sure where they are and what they are.</span></blockquote><div><br></div>I hope you're planning to look into this before submitting the change :)<div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 27, 2018 at 11:28 AM, Eugene Birukov via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">EugeneBi added a comment.<br>
<br>
Hmm... I never thought I can do that :)<br>
Anyway, with the change as it is now, LLDB would try to load executable in the sysroot, fail that, then open it without the sysroot.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D49685" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D49685</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>