[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