<div dir="ltr"><div class="gmail_extra">Thanks for the review!</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 15, 2013 at 6:21 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank" class="cremed">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":2tx" style="overflow:hidden">This is causing crashes in MSVC debug builds due to being an invalid<br>
iterator operation. Specifically, when I == ResultsList.begin(),<br>
attempting to decrement walks off the front of the iterator list.<br></div></blockquote><div><br></div><div>Doh, thanks. Fixed in r194857.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":2tx" style="overflow:hidden">
<br>
Once I correct the issue locally, I get several new failing unit tests:<br></div></blockquote><div><br></div><div>I think these are unrelated -- they aren't failing for me after fixing.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":2tx" style="overflow:hidden">
<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(189): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(193): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(189): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(193): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(189): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.<br>
E:\llvm\llvm\unittests\IR\ValueMapTest.cpp(193): error: Value of:<br>
Data.M->tryacquire()<br>
  Actual: true<br>
Expected: false<br>
Mutex should already be locked.</div></blockquote></div><br><br></div></div>