diff options
| author | Erich Keane <erich.keane@intel.com> | 2018-12-12 18:11:36 +0000 |
|---|---|---|
| committer | Erich Keane <erich.keane@intel.com> | 2018-12-12 18:11:36 +0000 |
| commit | 8e46c4b83f6c1788f78715ae2b66c35659b42538 (patch) | |
| tree | 6ebef22e944a723198a3d69aa18a8e17079ecc66 | |
| parent | a3e74a1b136acb5577ffff7882ba076de909869c (diff) | |
| download | bcm5719-llvm-8e46c4b83f6c1788f78715ae2b66c35659b42538.tar.gz bcm5719-llvm-8e46c4b83f6c1788f78715ae2b66c35659b42538.zip | |
Change CallGraph print to show the fully qualified name
CallGraph previously would just show the normal name of a function,
which gets really confusing when using it on large C++ projects. This
patch switches the printName call to a printQualifiedName, so that the
namespaces are included.
Change-Id: Ie086d863f6b2251be92109ea1b0946825b28b49a
llvm-svn: 348950
| -rw-r--r-- | clang/lib/Analysis/CallGraph.cpp | 2 | ||||
| -rw-r--r-- | clang/test/Analysis/debug-CallGraph.cpp | 11 |
2 files changed, 7 insertions, 6 deletions
diff --git a/clang/lib/Analysis/CallGraph.cpp b/clang/lib/Analysis/CallGraph.cpp index bac00680ffd..66a6f1a9bce 100644 --- a/clang/lib/Analysis/CallGraph.cpp +++ b/clang/lib/Analysis/CallGraph.cpp @@ -212,7 +212,7 @@ void CallGraph::viewGraph() const { void CallGraphNode::print(raw_ostream &os) const { if (const NamedDecl *ND = dyn_cast_or_null<NamedDecl>(FD)) - return ND->printName(os); + return ND->printQualifiedName(os); os << "< >"; } diff --git a/clang/test/Analysis/debug-CallGraph.cpp b/clang/test/Analysis/debug-CallGraph.cpp index 7f9ee4dc94f..1d6844fad94 100644 --- a/clang/test/Analysis/debug-CallGraph.cpp +++ b/clang/test/Analysis/debug-CallGraph.cpp @@ -51,6 +51,7 @@ void test_single_call() { do_nothing(); } +namespace SomeNS { template<typename T> void templ(T t) { ccc(); @@ -61,17 +62,17 @@ void templ<double>(double t) { eee(); } - void templUser() { templ(5); templ(5.5); } +} // CHECK:--- Call graph Dump --- -// CHECK-NEXT: {{Function: < root > calls: get5 add test_add mmm foo aaa < > bbb ddd ccc eee fff do_nothing test_single_call templ templ templUser $}} -// CHECK-NEXT: {{Function: templUser calls: templ templ $}} -// CHECK-NEXT: {{Function: templ calls: eee $}} -// CHECK-NEXT: {{Function: templ calls: ccc $}} +// CHECK-NEXT: {{Function: < root > calls: get5 add test_add mmm foo aaa < > bbb ddd ccc eee fff do_nothing test_single_call SomeNS::templ SomeNS::templ SomeNS::templUser $}} +// CHECK-NEXT: {{Function: SomeNS::templUser calls: SomeNS::templ SomeNS::templ $}} +// CHECK-NEXT: {{Function: SomeNS::templ calls: eee $}} +// CHECK-NEXT: {{Function: SomeNS::templ calls: ccc $}} // CHECK-NEXT: {{Function: test_single_call calls: do_nothing $}} // CHECK-NEXT: {{Function: do_nothing calls: $}} // CHECK-NEXT: {{Function: fff calls: eee $}} |
