This revision was automatically updated to reflect the committed changes.
Closed by commit rL288139: [change-namespace] fix non-calling function
references. (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D27208?vs=79558&id=79559#toc
Repository:
rL LLVM
https://revie
ioeric updated this revision to Diff 79558.
ioeric added a comment.
- Format code.
https://reviews.llvm.org/D27208
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: unittests/change-namespace/ChangeNa
ioeric updated this revision to Diff 79555.
ioeric marked 3 inline comments as done.
ioeric added a comment.
- Address comments.
https://reviews.llvm.org/D27208
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h
unittests/change-namespace/ChangeNamespaceTests.c
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
LGTM with few nits.
Comment at: change-namespace/ChangeNamespace.cpp:434
+assert(Func);
+const Decl *Context = Result.Nodes.getNodeAs("dc");
+assert(Context && "E
ioeric created this revision.
ioeric added a reviewer: hokein.
ioeric added a subscriber: cfe-commits.
https://reviews.llvm.org/D27208
Files:
change-namespace/ChangeNamespace.cpp
change-namespace/ChangeNamespace.h
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: unittests/change-