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