<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">On second thought, yes, that patch is a total think-o (right idea, wrong fix)..</div><div class=""><br class=""></div><div class="">It should probably just say</div><div class=""><br class=""></div><div class="">if (success)</div><div class=""> assert(what you said)</div><div class=""><br class=""></div><div class="">Better patch incoming.. and I wonder if we can in any way force ourselves into this scenario to test the sanity of any change in this area. Let me think about that</div><br class=""><div><blockquote type="cite" class=""><div class="">On Jan 8, 2015, at 10:45 AM, Zachary Turner <<a href="mailto:zturner@google.com" class="">zturner@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Ok so it looks like your patch doesn't fix the assert for me, but now looking more closely at your patch, I wonder if it's correct. As per the description, the purpose of your patch is to fix the case where success == false (which is what's happening to me), but you are adding a requirement to the assert that success be true. Shouldn't this be something like the following:<br class=""><div class=""><br class=""></div><div class=""><span style="line-height:19.7999992370605px" class="">assert (!need_compare_checksums || (!old_checksum.empty() && !m_value_checksum.empty()));</span><br class=""></div><div class=""><span style="line-height:19.7999992370605px" class=""><br class=""></span></div><div class=""><span style="line-height:19.7999992370605px" class=""><br class=""></span></div><br class=""><div class="gmail_quote">On Wed Jan 07 2015 at 5:50:44 PM Zachary Turner <<a href="mailto:zturner@google.com" class="">zturner@google.com</a>> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It just happens every time i run one of the tests on windows. It might be triggered because something earlier is broken, but I'll send you a stack trace or something tomorrow. Won't it be nice if/when we can debug windows core dumps from Mac? :P<br class=""><div class="gmail_quote">On Wed, Jan 7, 2015 at 5:21 PM Enrico Granata <<a href="mailto:egranata@apple.com" target="_blank" class="">egranata@apple.com</a>> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">If you end up with a reproducible case of the assertion firing, totally let me know - bugzilla or just an email</div><div class="">Hopefully it’s not so Windows-specific that I can’t get it to happen on Mac</div></div><div style="word-wrap:break-word" class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On Jan 7, 2015, at 4:33 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank" class="">zturner@google.com</a>> wrote:</div><br class=""><div class="">Cool, i think this assertion was actually firing on windows, making it very annoying to run the test suite. I hope this fixes that<br class=""><div class="gmail_quote">On Wed, Jan 7, 2015 at 4:30 PM Enrico Granata <<a href="mailto:egranata@apple.com" target="_blank" class="">egranata@apple.com</a>> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: enrico<br class="">
Date: Wed Jan 7 18:29:12 2015<br class="">
New Revision: 225418<br class="">
<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=225418&view=rev" target="_blank" class="">http://llvm.org/viewvc/llvm-<u class=""></u>pr<u class=""></u>oject?rev=225418&view=rev</a><br class="">
Log:<br class="">
Fix a problem where a ValueObject could fail to update itself, but since it was previously valid, we'd have an old checksum to compare aginst no new checksum (because failure to update), and assert() and die. Fix the problem by only caring about this assertion logic if updates succeed<br class="">
<br class="">
Modified:<br class="">
lldb/trunk/source/Core/<u class=""></u>ValueOb<u class=""></u>ject.cpp<br class="">
<br class="">
Modified: lldb/trunk/source/Core/<u class=""></u>ValueOb<u class=""></u>ject.cpp<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Core/ValueObject.cpp?rev=225418&r1=225417&r2=225418&view=diff" target="_blank" class="">http://llvm.org/viewvc/llvm-<u class=""></u>pr<u class=""></u>oject/lldb/trunk/source/<u class=""></u>Core/<u class=""></u>ValueObject.cpp?rev=<u class=""></u>225418&r1=<u class=""></u>225417&r2=225418&<u class=""></u>view=diff</a><br class="">
==============================<u class=""></u><u class=""></u>==============================<u class=""></u><u class=""></u>==================<br class="">
--- lldb/trunk/source/Core/<u class=""></u>ValueOb<u class=""></u>ject.cpp (original)<br class="">
+++ lldb/trunk/source/Core/<u class=""></u>ValueOb<u class=""></u>ject.cpp Wed Jan 7 18:29:12 2015<br class="">
@@ -250,7 +250,7 @@ ValueObject::<u class=""></u>UpdateValueIfNeed<u class=""></u>ed (bool u<br class="">
m_value_checksum.clear();<br class="">
}<br class="">
<br class="">
- assert (old_checksum.empty() == !need_compare_checksums);<br class="">
+ assert (success && (old_checksum.empty() == !need_compare_checksums));<br class="">
<br class="">
if (first_update)<br class="">
SetValueDidChange (false);<br class="">
<br class="">
<br class="">
______________________________<u class=""></u><u class=""></u>_________________<br class="">
lldb-commits mailing list<br class="">
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank" class="">lldb-commits@cs.uiuc.edu</a><br class="">
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank" class="">http://lists.cs.uiuc.edu/<u class=""></u>mailm<u class=""></u>an/listinfo/lldb-commits</a><br class="">
</blockquote></div>
</div></blockquote></div><br class=""></div><div style="word-wrap:break-word" class=""><div class="">
<div style="letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; word-wrap: break-word;" class=""><div style="letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; word-wrap: break-word;" class=""><div style="letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; word-wrap: break-word;" class=""><div style="letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; word-wrap: break-word;" class=""><div style="letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; word-wrap: break-word;" class=""><div class="">Thanks,</div><div class=""><i class="">- Enrico</i><br class="">📩 egranata@<font color="#ff2600" class=""></font>.com ☎️ 27683</div><div class=""><br class=""></div></div></div></div></div></div><br class=""><br class="">
</div>
<br class=""></div></blockquote></div></blockquote></div>
</div></blockquote></div><br class=""><div class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Thanks,</div><div class=""><i class="">- Enrico</i><br class="">📩 egranata@<font color="#ff2600" class=""></font>.com ☎️ 27683</div><div class=""><br class=""></div></div></div></div></div></div><br class="Apple-interchange-newline"><br class="Apple-interchange-newline">
</div>
<br class=""></body></html>