<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Sorry about the breakage, and thanks for working on a patch Chaoren.<br>
Hopefully Marianne will revise her merge request if this fix is
still important for her.<br>
<br>
<div class="moz-cite-prefix">On 28/02/2016 00:06, Todd Fiala wrote:<br>
</div>
<blockquote
cite="mid:EA7BA33F-F398-4AFC-AEA4-F76878E0470D@gmail.com"
type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<div>I brought the state of the lines changed there to what they
were prior to the change. If you adjusted those same lines, then
yes that got undone.</div>
<div id="AppleMailSignature"><br>
</div>
<div id="AppleMailSignature">We were failing different tests in
those files. Your change may have fixed the new issues on
Linux, but did not address the new failures on OS X. The OS X
bot went green after the revert.</div>
<div id="AppleMailSignature"><br>
-Todd</div>
<div><br>
On Feb 27, 2016, at 4:01 PM, Chaoren Lin <<a
moz-do-not-send="true" href="mailto:chaorenl@google.com"><a class="moz-txt-link-abbreviated" href="mailto:chaorenl@google.com">chaorenl@google.com</a></a>>
wrote:<br>
<br>
</div>
<blockquote type="cite">
<div>
<div dir="ltr">I thought I fixed it with <a
moz-do-not-send="true"
href="http://reviews.llvm.org/D17658" target="_blank"><a class="moz-txt-link-freetext" href="http://reviews.llvm.org/D17658">http://reviews.llvm.org/D17658</a></a>.
Did you revert that as well?
<div class="gmail_extra"><br>
<div class="gmail_quote">On Sat, Feb 27, 2016 at 3:14 PM,
Todd Fiala <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:todd.fiala@gmail.com" target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a></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">Hi all,
<div><br>
</div>
<div>The new assert was failing on the OS X testbot
here:</div>
<div><a moz-do-not-send="true"
href="http://lab.llvm.org:8080/green/job/lldb_build_test/"
target="_blank">http://lab.llvm.org:8080/green/job/lldb_build_test/</a><br>
</div>
<div><br>
</div>
<div>The nag mail didn't make it to you since we
have it only fire on the transition from good
build to bad build, and Tamas had the privilege of
earning that with a minor Xcode breakage just
shortly before this issue showed up. However,
that bot as been broken since this change went in.</div>
<div><br>
</div>
<div>I reverted it in r262156.</div>
<div><br>
</div>
<div>Feel free to reapply if you have a suggested
fix for the test failure introduced.</div>
<div><br>
</div>
<div>Thanks!</div>
<div><br>
</div>
<div>-Todd</div>
</div>
<div class="gmail_extra">
<div>
<div><br>
<div class="gmail_quote">On Fri, Feb 26, 2016 at
12:08 PM, Chaoren Lin via lldb-commits <span
dir="ltr"><<a moz-do-not-send="true"
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">
<div dir="ltr">Hmm. Weird. That assert is
failing on Linux: <a
moz-do-not-send="true"
href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/11833"
target="_blank"><a class="moz-txt-link-freetext" href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/11833">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-cmake/builds/11833</a></a></div>
<div>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Fri, Feb
26, 2016 at 9:40 AM, Aidan Dodds via
lldb-commits <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:lldb-commits@lists.llvm.org"
target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a></a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="margin:0 0 0
.8ex;border-left:1px #ccc
solid;padding-left:1ex">Author:
aidandodds<br>
Date: Fri Feb 26 11:40:50 2016<br>
New Revision: 262041<br>
<br>
URL: <a moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project?rev=262041&view=rev"
rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=262041&view=rev</a><br>
Log:<br>
Fix bug with register values byte
order in expression evaluation.<br>
<br>
The evaluation of expressions
containing register values was
broken for targets for which
endianness differs from host.<br>
<br>
Committed on behalf of: mamai <<a
moz-do-not-send="true"
href="mailto:marianne.mailhot.sarrasin@gmail.com"
target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:marianne.mailhot.sarrasin@gmail.com">marianne.mailhot.sarrasin@gmail.com</a></a>><br>
<br>
Differential revision: <a
moz-do-not-send="true"
href="http://reviews.llvm.org/D17167"
rel="noreferrer" target="_blank"><a class="moz-txt-link-freetext" href="http://reviews.llvm.org/D17167">http://reviews.llvm.org/D17167</a></a><br>
<br>
Modified:<br>
lldb/trunk/source/Expression/Materializer.cpp<br>
<br>
Modified:
lldb/trunk/source/Expression/Materializer.cpp<br>
URL: <a moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Expression/Materializer.cpp?rev=262041&r1=262040&r2=262041&view=diff"
rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Expression/Materializer.cpp?rev=262041&r1=262040&r2=262041&view=diff</a><br>
==============================================================================<br>
---
lldb/trunk/source/Expression/Materializer.cpp
(original)<br>
+++
lldb/trunk/source/Expression/Materializer.cpp
Fri Feb 26 11:40:50 2016<br>
@@ -26,6 +26,7 @@<br>
#include
"lldb/Target/StackFrame.h"<br>
#include "lldb/Target/Target.h"<br>
#include "lldb/Target/Thread.h"<br>
+#include
"lldb/Utility/LLDBAssert.h"<br>
<br>
using namespace lldb_private;<br>
<br>
@@ -1275,9 +1276,14 @@ public:<br>
m_register_contents.reset(new
DataBufferHeap(register_data.GetDataStart(),
register_data.GetByteSize()));<br>
<br>
Error write_error;<br>
-<br>
-
map.WriteMemory(load_addr,
register_data.GetDataStart(),
register_data.GetByteSize(),
write_error);<br>
-<br>
+<br>
+ Scalar scalar;<br>
+
reg_value.GetScalarValue(scalar);<br>
+<br>
+
lldbassert(scalar.GetByteSize() ==
register_data.GetByteSize());<br>
+<br>
+
map.WriteScalarToMemory(load_addr,
scalar, scalar.GetByteSize(),
write_error);<br>
+<br>
if
(!write_error.Success())<br>
{<br>
err.SetErrorStringWithFormat("couldn't
write the contents of register %s:
%s", <a moz-do-not-send="true"
href="http://m_register_info.name"
rel="noreferrer" target="_blank">m_register_info.name</a>,
write_error.AsCString());<br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a moz-do-not-send="true"
href="mailto:lldb-commits@lists.llvm.org"
target="_blank">lldb-commits@lists.llvm.org</a><br>
<a moz-do-not-send="true"
href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits"
rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote>
</div>
<br>
</div>
</div>
</div>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a moz-do-not-send="true"
href="mailto:lldb-commits@lists.llvm.org"
target="_blank">lldb-commits@lists.llvm.org</a><br>
<a moz-do-not-send="true"
href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits"
rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
</div>
</div>
<span><font color="#888888">-- <br>
<div>
<div dir="ltr">-Todd</div>
</div>
</font></span></div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</blockquote>
</blockquote>
<br>
</body>
</html>