[PATCH] D24800: Merge conflicting replacements when they are order-independent.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 28 00:44:07 PDT 2016


klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.

lg


================
Comment at: lib/Tooling/Core/Replacement.cpp:179-181
@@ +178,5 @@
+// `R` and `Replaces` are order-independent if applying them in either order
+// has the same effect, so we need to compare replacements associated to
+// applying them in either order.
+llvm::Expected<Replacements>
+Replacements::mergeIfOrderIndependent(const Replacement &R) const {
----------------
Much better. Thanks.


https://reviews.llvm.org/D24800





More information about the cfe-commits mailing list