maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Revert r326723: Make STATISTIC() values available programmatically 

Despite building cleanly on my machine in three separate configs, it's failing on pretty much all bots due to missing includes among other things. Investigating.

llvm-svn: 326726
This commit is contained in:
Daniel Sanders 2018-03-05 17:52:43 +00:00
parent 1578a0a55d
commit 7612f85df5
6 changed files with 7 additions and 154 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/cmake/modules/HandleLLVMOptions.cmake
View File

@ -881,7 +881,3 @@ function(get_compile_definitions)
set(LLVM_DEFINITIONS "${result}" PARENT_SCOPE)
endfunction()
get_compile_definitions()
# The default for LLVM_ENABLE_STATS depends on whether NDEBUG is defined or not.
# LLVM_ENABLE_ASSERTIONS controls that so re-use it as the default.
option(LLVM_ENABLE_STATS "Enable statistics collection" ${LLVM_ENABLE_ASSERTIONS})

15
llvm/include/llvm/ADT/Statistic.h
View File

@ -26,7 +26,6 @@
#ifndef LLVM_ADT_STATISTIC_H
#define LLVM_ADT_STATISTIC_H
#include "llvm/Config/llvm-config.h"
#include "llvm/Support/Compiler.h"
#include <atomic>
#include <memory>
@ -35,7 +34,6 @@ namespace llvm {
class raw_ostream;
class raw_fd_ostream;
class StringRef;
class Statistic {
public:
@ -62,7 +60,7 @@ public:
// Allow use of this class as the value itself.
operator unsigned() const { return getValue(); }
#if LLVM_ENABLE_STATS
#if !defined(NDEBUG) || defined(LLVM_ENABLE_STATS)
const Statistic &operator=(unsigned Val) {
Value.store(Val, std::memory_order_relaxed);
return init();
@ -144,7 +142,7 @@ public:
void updateMax(unsigned V) {}
#endif // LLVM_ENABLE_STATS
#endif // !defined(NDEBUG) || defined(LLVM_ENABLE_STATS)
protected:
Statistic &init() {
@ -182,15 +180,6 @@ void PrintStatistics(raw_ostream &OS);
/// PrintStatisticsJSON().
void PrintStatisticsJSON(raw_ostream &OS);
/// \brief Get the statistics. This can be used to look up the value of
/// statistics without needing to parse JSON.
///
/// This function does not prevent statistics being updated by other threads
/// during it's execution. It will return the value at the point that it is
/// read. However, it will prevent new statistics from registering until it
/// completes.
const std::vector<std::pair<StringRef, unsigned>> GetStatistics();
} // end namespace llvm
#endif // LLVM_ADT_STATISTIC_H

5
llvm/include/llvm/Config/llvm-config.h.cmake
View File

@ -77,9 +77,4 @@
/* LLVM version string */
#define LLVM_VERSION_STRING "${PACKAGE_VERSION}"
/* Whether LLVM records statistics for use with GetStatistics(),
* PrintStatistics() or PrintStatisticsJSON()
*/
#cmakedefine01 LLVM_ENABLE_STATS
#endif

30
llvm/lib/Support/Statistic.cpp
View File

@ -52,14 +52,11 @@ static bool Enabled;
static bool PrintOnExit;
namespace {
/// This class is used in a ManagedStatic so that it is created on demand (when
/// the first statistic is bumped) and destroyed only when llvm_shutdown is
/// called. We print statistics from the destructor.
/// This class is also used to look up statistic values from applications that
/// use LLVM.
/// StatisticInfo - This class is used in a ManagedStatic so that it is created
/// on demand (when the first statistic is bumped) and destroyed only when
/// llvm_shutdown is called. We print statistics from the destructor.
class StatisticInfo {
std::vector<const Statistic*> Stats;
friend void llvm::PrintStatistics();
friend void llvm::PrintStatistics(raw_ostream &OS);
friend void llvm::PrintStatisticsJSON(raw_ostream &OS);
@ -67,22 +64,14 @@ class StatisticInfo {
/// Sort statistics by debugtype,name,description.
void sort();
public:
using const_iterator = std::vector<const Statistic *>::const_iterator;
StatisticInfo();
~StatisticInfo();
void addStatistic(const Statistic *S) {
Stats.push_back(S);
}
const_iterator begin() const { return Stats.begin(); }
const_iterator end() const { return Stats.end(); }
iterator_range<const_iterator> statistics() const {
return {begin(), end()};
}
};
} // end anonymous namespace
}
static ManagedStatic<StatisticInfo> StatInfo;
static ManagedStatic<sys::SmartMutex<true> > StatLock;
@ -191,7 +180,7 @@ void llvm::PrintStatisticsJSON(raw_ostream &OS) {
}
void llvm::PrintStatistics() {
#if LLVM_ENABLE_STATS
#if !defined(NDEBUG) || defined(LLVM_ENABLE_STATS)
StatisticInfo &Stats = *StatInfo;
// Statistics not enabled?
@ -216,12 +205,3 @@ void llvm::PrintStatistics() {
}
#endif
}
const std::vector<std::pair<StringRef, unsigned>> llvm::GetStatistics() {
sys::SmartScopedLock<true> Reader(*StatLock);
std::vector<std::pair<StringRef, unsigned>> ReturnStats;
for (const auto &Stat : StatInfo->statistics())
ReturnStats.emplace_back(Stat->getName(), Stat->getValue());
return ReturnStats;
}

1
llvm/unittests/ADT/CMakeLists.txt
View File

@ -56,7 +56,6 @@ set(ADTSources
SparseBitVectorTest.cpp
SparseMultiSetTest.cpp
SparseSetTest.cpp
StatisticTest.cpp
StringExtrasTest.cpp
StringMapTest.cpp
StringRefTest.cpp

106
llvm/unittests/ADT/StatisticTest.cpp
View File

@ -1,106 +0,0 @@
//===- llvm/unittest/ADT/StatisticTest.cpp - Statistic unit tests ---------===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
#include "llvm/ADT/Statistic.h"
#include "llvm/Support/raw_ostream.h"
#include "gtest/gtest.h"
using namespace llvm;
namespace {
#define DEBUG_TYPE "unittest"
STATISTIC(Counter, "Counts things");
STATISTIC(Counter2, "Counts other things");
TEST(StatisticTest, Count) {
EnableStatistics();
Counter = 0;
EXPECT_EQ(Counter, 0u);
Counter++;
Counter++;
#if LLVM_ENABLE_STATS
EXPECT_EQ(Counter, 2u);
#else
EXPECT_EQ(Counter, 0u);
#endif
}
TEST(StatisticTest, Assign) {
EnableStatistics();
Counter = 2;
#if LLVM_ENABLE_STATS
EXPECT_EQ(Counter, 2u);
#else
EXPECT_EQ(Counter, 0u);
#endif
}
TEST(StatisticTest, API) {
EnableStatistics();
Counter = 0;
EXPECT_EQ(Counter, 0u);
Counter++;
Counter++;
#if LLVM_ENABLE_STATS
EXPECT_EQ(Counter, 2u);
#else
EXPECT_EQ(Counter, 0u);
#endif
#if LLVM_ENABLE_STATS
const auto Range1 = GetStatistics();
EXPECT_NE(Range1.begin(), Range1.end());
EXPECT_EQ(Range1.begin() + 1, Range1.end());
Optional<std::pair<StringRef, unsigned>> S1;
Optional<std::pair<StringRef, unsigned>> S2;
for (const auto &S : Range1) {
if (std::string(S.first) == "Counter")
S1 = S;
if (std::string(S.first) == "Counter2")
S2 = S;
}
EXPECT_NE(S1.hasValue(), false);
EXPECT_EQ(S2.hasValue(), false);
// Counter2 will be registered when it's first touched.
Counter2++;
const auto Range2 = GetStatistics();
EXPECT_NE(Range2.begin(), Range2.end());
EXPECT_EQ(Range2.begin() + 2, Range2.end());
S1 = None;
S2 = None;
for (const auto &S : Range2) {
if (std::string(S.first) == "Counter")
S1 = S;
if (std::string(S.first) == "Counter2")
S2 = S;
}
EXPECT_NE(S1.hasValue(), false);
EXPECT_NE(S2.hasValue(), false);
EXPECT_EQ(S1->first, "Counter");
EXPECT_EQ(S1->second, 2u);
EXPECT_EQ(S2->first, "Counter2");
EXPECT_EQ(S2->second, 1u);
#else
Counter2++;
auto &Range = GetStatistics();
EXPECT_EQ(Range.begin(), Range.end());
#endif
}
} // end anonymous namespace