<div dir="ltr">Which was the first patch that passed the buildbots? Could you point/mention the commit revision? & then the commit revision that failed? I hadn't realized a version had been committed before.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jul 2, 2018 at 3:03 PM Paul Semel via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">paulsemel added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D48271#1150086" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48271#1150086</a>, @dblaikie wrote:<br>
<br>
> In <a href="https://reviews.llvm.org/D48271#1149023" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48271#1149023</a>, @paulsemel wrote:<br>
><br>
> > I got a build fail here : <a href="http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/10648" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/10648</a><br>
> >  As for now, I still don't know why I'm getting this error, but I guessed it was because I was passing the raw_stream..<br>
> >  This should fix this, I hope<br>
><br>
><br>
> I'd guess it's pretty unlikely that it's the passing of a raw_ostream by reference - that's a pretty common thing. To my mind, the code that looks more likely to be crashing there is that maybe the StringRef is out-of-bounds and the range-based for loop is running off the end of the underlying buffer.<br>
><br>
> Have you tried running the test under valgrind? asan? msan? things like that. You might be able to reproduce/identify the crash on your machine so you can validate a fix rather than throwing it at the buildbots again.<br>
<br>
<br>
Well, I ran it for the first patch. This one, I'm just refactoring, nothing has changed for the bounds right ?<br>
Of the first patch passed the buildbots, that means there was no problem about bounds (and as I said, I ran asan on it).<br>
I will do it again tomorrow, maybe the but was not trigger before and is now, but I honestly doubt it :)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D48271" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48271</a><br>
<br>
<br>
<br>
</blockquote></div>