Thank you. I think I have fixed the issue in r248375.

On Wed, Sep 23, 2015 at 12:25 PM, NAKAMURA Takumi <geek4ci...@gmail.com>
wrote:

> chapuni added subscribers: cfe-commits, chapuni.
>
> /cfe/trunk/lib/Tooling/Core/Replacement.cpp:408 FirstI may be in the end()
> when SecondI != end(). Then FirstI->getOffset() is invalid.
>
> I suggest to split the loop into
>   - FirstI != First.end() && SecondI != Second.end();
>   - Tail-merge (either is end())
>
> Users:
>   djasper (Author)
>
> http://reviews.llvm.org/rL248367
>
>
>
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to