[PATCH] D27982: [change-namespace] do not fix calls to overloaded operator functions.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 20 06:42:57 PST 2016


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

LGTM.



================
Comment at: unittests/change-namespace/ChangeNamespaceTests.cpp:578
 
+// FIXME: function calls to overloaded operators are not fixed now even if they
+// are referenced by qualified names.
----------------
Maybe also add this case in the test although it is not fixed yet.


================
Comment at: unittests/change-namespace/ChangeNamespaceTests.cpp:592
+      "  A x, y;\n"
+      "  auto f = operator<;"
+      "  return (x == y) && (x < y) && (operator<(x, y));"
----------------
missing a `\n`, the same below.


https://reviews.llvm.org/D27982





More information about the cfe-commits mailing list