<div dir="ltr">Hi,<div><br></div><div>Is the work that you are currently doing the reason why the lldb build on i386 is failing?<br><div><a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake</a><br></div></div><div><a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/20099">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/20099</a><br></div><div><br></div><div>Dimitar</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 29, 2016 at 5:30 AM, Jason Molenda via lldb-commits <span dir="ltr"><<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Good suggestions, thanks.  I'll fix those when I commit the 32-bit version of the same test.<br>
<span class="HOEnZb"><font color="#888888"><br>
J<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Sep 28, 2016, at 9:28 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
><br>
><br>
><br>
> On Wed, Sep 28, 2016 at 9:10 PM Jason Molenda via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br>
><br>
> +  EXPECT_TRUE(regloc.GetOffset() == -8);<br>
> This should be<br>
><br>
> EXPECT_EQ(-8, regloc.GetOffset());<br>
><br>
> That way if it fails, you'll get a handy error message that says:<br>
><br>
> Expected: -8<br>
> Actual: -7<br>
><br>
> If you use EXPECT_TRUE, it's not going to tell you the actual value.  The same goes for many other places in the file.  Note that you're supposed to put the expected value *first*.  The test is the same either way obviously, but it affects the printing of the above message.<br>
><br>
> +<br>
> +  // these could be set to IsSame and be valid -- meaning that the<br>
> +  // register value is the same as the caller's -- but I'd rather<br>
> +  // they not be mentioned at all.<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_rbp, regloc) == false);<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r15, regloc) == false);<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r14, regloc) == false);<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r13, regloc) == false);<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_r12, regloc) == false);<br>
> +  EXPECT_TRUE(row_sp-><wbr>GetRegisterInfo(k_rbx, regloc) == false);<br>
> If you're using EXPECT_TRUE and EXPECT_FALSE, I think it's more intuitive to not use the comparison operator.  The above is just<br>
><br>
> EXPECT_FALSE(row_sp-><wbr>GetRegisterInfo(k_rbx, regloc));<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-commits</a><br>
</div></div></blockquote></div><br></div>