[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