This revision was automatically updated to reflect the committed changes.
Closed by commit rL348584: [CTU] Add statistics (authored by martong, committed
by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D55133?vs=176655&id=177173#toc
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55133/new/
https://reviews.llvm.org/D55133
Files:
cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
Index: cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
===================================================================
--- cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
+++ cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
@@ -21,6 +21,7 @@
#include "clang/Frontend/TextDiagnosticPrinter.h"
#include "clang/Index/USRGeneration.h"
#include "llvm/ADT/Triple.h"
+#include "llvm/ADT/Statistic.h"
#include "llvm/Support/ErrorHandling.h"
#include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/Path.h"
@@ -32,6 +33,15 @@
namespace cross_tu {
namespace {
+#define DEBUG_TYPE "CrossTranslationUnit"
+STATISTIC(NumGetCTUCalled, "The # of getCTUDefinition function called");
+STATISTIC(
+ NumNotInOtherTU,
+ "The # of getCTUDefinition called but the function is not in any other
TU");
+STATISTIC(NumGetCTUSuccess,
+ "The # of getCTUDefinition successfully returned the "
+ "requested function's body");
+
// FIXME: This class is will be removed after the transition to llvm::Error.
class IndexErrorCategory : public std::error_category {
public:
@@ -151,6 +161,7 @@
StringRef CrossTUDir,
StringRef IndexName) {
assert(!FD->hasBody() && "FD has a definition in current translation unit!");
+ ++NumGetCTUCalled;
const std::string LookupFnName = getLookupName(FD);
if (LookupFnName.empty())
return llvm::make_error<IndexError>(
@@ -216,8 +227,10 @@
}
auto It = FunctionFileMap.find(LookupName);
- if (It == FunctionFileMap.end())
+ if (It == FunctionFileMap.end()) {
+ ++NumNotInOtherTU;
return
llvm::make_error<IndexError>(index_error_code::missing_definition);
+ }
StringRef ASTFileName = It->second;
auto ASTCacheEntry = FileASTUnitMap.find(ASTFileName);
if (ASTCacheEntry == FileASTUnitMap.end()) {
@@ -250,6 +263,7 @@
cast<FunctionDecl>(Importer.Import(const_cast<FunctionDecl *>(FD)));
assert(ToDecl->hasBody());
assert(FD->hasBody() && "Functions already imported should have body.");
+ ++NumGetCTUSuccess;
return ToDecl;
}
Index: cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
===================================================================
--- cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
+++ cfe/trunk/lib/CrossTU/CrossTranslationUnit.cpp
@@ -21,6 +21,7 @@
#include "clang/Frontend/TextDiagnosticPrinter.h"
#include "clang/Index/USRGeneration.h"
#include "llvm/ADT/Triple.h"
+#include "llvm/ADT/Statistic.h"
#include "llvm/Support/ErrorHandling.h"
#include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/Path.h"
@@ -32,6 +33,15 @@
namespace cross_tu {
namespace {
+#define DEBUG_TYPE "CrossTranslationUnit"
+STATISTIC(NumGetCTUCalled, "The # of getCTUDefinition function called");
+STATISTIC(
+ NumNotInOtherTU,
+ "The # of getCTUDefinition called but the function is not in any other TU");
+STATISTIC(NumGetCTUSuccess,
+ "The # of getCTUDefinition successfully returned the "
+ "requested function's body");
+
// FIXME: This class is will be removed after the transition to llvm::Error.
class IndexErrorCategory : public std::error_category {
public:
@@ -151,6 +161,7 @@
StringRef CrossTUDir,
StringRef IndexName) {
assert(!FD->hasBody() && "FD has a definition in current translation unit!");
+ ++NumGetCTUCalled;
const std::string LookupFnName = getLookupName(FD);
if (LookupFnName.empty())
return llvm::make_error<IndexError>(
@@ -216,8 +227,10 @@
}
auto It = FunctionFileMap.find(LookupName);
- if (It == FunctionFileMap.end())
+ if (It == FunctionFileMap.end()) {
+ ++NumNotInOtherTU;
return llvm::make_error<IndexError>(index_error_code::missing_definition);
+ }
StringRef ASTFileName = It->second;
auto ASTCacheEntry = FileASTUnitMap.find(ASTFileName);
if (ASTCacheEntry == FileASTUnitMap.end()) {
@@ -250,6 +263,7 @@
cast<FunctionDecl>(Importer.Import(const_cast<FunctionDecl *>(FD)));
assert(ToDecl->hasBody());
assert(FD->hasBody() && "Functions already imported should have body.");
+ ++NumGetCTUSuccess;
return ToDecl;
}
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
