This revision was automatically updated to reflect the committed changes.
Closed by commit rL290176: [change-namespace] do not fix calls to overloaded
operator functions. (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D27982?vs=82104=82105#toc
Repository:
rL LLVM
ioeric updated this revision to Diff 82104.
ioeric marked 2 inline comments as done.
ioeric added a comment.
- Address comments
https://reviews.llvm.org/D27982
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h
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
ioeric updated this revision to Diff 82098.
ioeric added a comment.
- minor fix.
https://reviews.llvm.org/D27982
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h
unittests/change-namespace/ChangeNamespaceTests.cpp
Index:
ioeric created this revision.
ioeric added a reviewer: hokein.
ioeric added a subscriber: cfe-commits.
Also make sure one function reference is only processed once.
https://reviews.llvm.org/D27982
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h