djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land.
Looks good. ================ Comment at: include/clang/Tooling/Core/Replacement.h:172 @@ -169,1 +171,3 @@ + /// - aren't both inserts at the same code location. + /// Note: two insertions at the same offset are considered order-dependent. /// Replacements with offset UINT_MAX are special - we do not detect conflicts ---------------- Remove the Note line and just add "(would be order-dependent)" to the line above. https://reviews.llvm.org/D24515 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits