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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits