<div dir="ltr">Thank you. I think I have fixed the issue in r248375.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 12:25 PM, NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chapuni added subscribers: cfe-commits, chapuni.<br>
<br>
/cfe/trunk/lib/Tooling/Core/Replacement.cpp:408 FirstI may be in the end() when SecondI != end(). Then FirstI->getOffset() is invalid.<br>
<br>
I suggest to split the loop into<br>
  - FirstI != First.end() && SecondI != Second.end();<br>
  - Tail-merge (either is end())<br>
<div class="HOEnZb"><div class="h5"><br>
Users:<br>
  djasper (Author)<br>
<br>
<a href="http://reviews.llvm.org/rL248367" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL248367</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>