[ms-inline asm] Make the comment a bit more verbose.
[oota-llvm.git] / lib / Analysis / ProfileDataLoader.cpp
index 01b50384f9473a869579def0ddcb93b29a8b56d6..d7f444b4b6d7f12c94395b90d39d69d9b3f7bd17 100644 (file)
 //
 //===----------------------------------------------------------------------===//
 
+#include "llvm/Analysis/ProfileDataLoader.h"
 #include "llvm/ADT/ArrayRef.h"
 #include "llvm/ADT/OwningPtr.h"
-#include "llvm/Module.h"
-#include "llvm/InstrTypes.h"
-#include "llvm/Analysis/ProfileDataLoader.h"
 #include "llvm/Analysis/ProfileDataTypes.h"
+#include "llvm/IR/InstrTypes.h"
+#include "llvm/IR/Module.h"
 #include "llvm/Support/raw_ostream.h"
 #include "llvm/Support/system_error.h"
 #include <cstdio>
 #include <cstdlib>
 using namespace llvm;
 
-namespace llvm {
-
-template<>
-char ProfileDataT<Function,BasicBlock>::ID = 0;
-
-raw_ostream& operator<<(raw_ostream &O, const Function *F) {
-  return O << F->getName();
-}
-
-raw_ostream& operator<<(raw_ostream &O, const BasicBlock *BB) {
-  return O << BB->getName();
-}
-
-raw_ostream& operator<<(raw_ostream &O, std::pair<const BasicBlock *,
-                        const BasicBlock *> E) {
+raw_ostream &llvm::operator<<(raw_ostream &O, std::pair<const BasicBlock *,
+                                                        const BasicBlock *> E) {
   O << "(";
 
   if (E.first)
-    O << E.first;
+    O << E.first->getName();
   else
     O << "0";
 
   O << ",";
 
   if (E.second)
-    O << E.second;
+    O << E.second->getName();
   else
     O << "0";
 
   return O << ")";
 }
 
-} // namespace llvm
-
-/// ByteSwap - Byteswap 'Var'.  Required when the compiler host and target have
-/// different endianness.
-static inline unsigned ByteSwap(unsigned Var) {
-  return ((Var & (255U<< 0U)) << 24U) |
-         ((Var & (255U<< 8U)) <<  8U) |
-         ((Var & (255U<<16U)) >>  8U) |
-         ((Var & (255U<<24U)) >> 24U);
-}
-
 /// AddCounts - Add 'A' and 'B', accounting for the fact that the value of one
 /// (or both) may not be defined.
 static unsigned AddCounts(unsigned A, unsigned B) {
@@ -75,32 +51,25 @@ static unsigned AddCounts(unsigned A, unsigned B) {
   if (A == ProfileDataLoader::Uncounted) return B;
   if (B == ProfileDataLoader::Uncounted) return A;
 
-  // Saturate to the maximum storable value.  This could change taken/nottaken
-  // ratios, but is presumably better than wrapping and thus potentially
-  // inverting ratios.
-  uint64_t tmp = (uint64_t)A + (uint64_t)B;
-  if (tmp > (uint64_t)ProfileDataLoader::MaxCount)
-    tmp = ProfileDataLoader::MaxCount;
-  return (unsigned)tmp;
+  return A + B;
 }
 
 /// ReadProfilingData - Load 'NumEntries' items of type 'T' from file 'F'
-template <typename T, unsigned N>
+template <typename T>
 static void ReadProfilingData(const char *ToolName, FILE *F,
-                              SmallVector<T, N> &Data, size_t NumEntries) {
+                              T *Data, size_t NumEntries) {
   // Read in the block of data...
-  if (fread(&Data[0], sizeof(T), NumEntries, F) != NumEntries) {
-    report_fatal_error(std::string(ToolName) + ": Profiling data truncated");
-  }
+  if (fread(Data, sizeof(T), NumEntries, F) != NumEntries)
+    report_fatal_error(Twine(ToolName) + ": Profiling data truncated");
 }
 
 /// ReadProfilingNumEntries - Read how many entries are in this profiling data
 /// packet.
 static unsigned ReadProfilingNumEntries(const char *ToolName, FILE *F,
                                         bool ShouldByteSwap) {
-  SmallVector<unsigned, 1> NumEntries(1);
-  ReadProfilingData<unsigned, 1>(ToolName, F, NumEntries, 1);
-  return ShouldByteSwap ? ByteSwap(NumEntries[0]) : NumEntries[0];
+  unsigned Entry;
+  ReadProfilingData<unsigned>(ToolName, F, &Entry, 1);
+  return ShouldByteSwap ? ByteSwap_32(Entry) : Entry;
 }
 
 /// ReadProfilingBlock - Read the number of entries in the next profiling data
@@ -113,16 +82,17 @@ static void ReadProfilingBlock(const char *ToolName, FILE *F,
 
   // Read in the data.
   SmallVector<unsigned, 8> TempSpace(NumEntries);
-  ReadProfilingData<unsigned, 8>(ToolName, F, TempSpace, (size_t)NumEntries);
+  ReadProfilingData<unsigned>(ToolName, F, TempSpace.data(), NumEntries);
 
   // Make sure we have enough space ...
   if (Data.size() < NumEntries)
     Data.resize(NumEntries, ProfileDataLoader::Uncounted);
 
   // Accumulate the data we just read into the existing data.
-  for (unsigned i = 0; i < NumEntries; ++i)
-    Data[i] = AddCounts(ShouldByteSwap ? ByteSwap(TempSpace[i]) : TempSpace[i],
-                        Data[i]);
+  for (unsigned i = 0; i < NumEntries; ++i) {
+    unsigned Entry = ShouldByteSwap ? ByteSwap_32(TempSpace[i]) : TempSpace[i];
+    Data[i] = AddCounts(Entry, Data[i]);
+  }
 }
 
 /// ReadProfilingArgBlock - Read the command line arguments that the progam was
@@ -137,14 +107,13 @@ static void ReadProfilingArgBlock(const char *ToolName, FILE *F,
   // the nearest 4-byte multiple.
   SmallVector<char, 8> Args(ArgLength+4);
   if (ArgLength)
-    ReadProfilingData<char, 8>(ToolName, F, Args, (ArgLength+3) & ~3);
+    ReadProfilingData<char>(ToolName, F, Args.data(), (ArgLength+3) & ~3);
 
   // Store the arguments.
   CommandLines.push_back(std::string(&Args[0], &Args[ArgLength]));
 }
 
 const unsigned ProfileDataLoader::Uncounted = ~0U;
-const unsigned ProfileDataLoader::MaxCount = ~0U - 1U;
 
 /// ProfileDataLoader ctor - Read the specified profiling data file, reporting
 /// a fatal error if the file is invalid or broken.
@@ -153,7 +122,7 @@ ProfileDataLoader::ProfileDataLoader(const char *ToolName,
   : Filename(Filename) {
   FILE *F = fopen(Filename.c_str(), "rb");
   if (F == 0)
-    report_fatal_error(std::string(ToolName) + ": Error opening '" +
+    report_fatal_error(Twine(ToolName) + ": Error opening '" +
                        Filename + "': ");
 
   // Keep reading packets until we run out of them.
@@ -164,7 +133,7 @@ ProfileDataLoader::ProfileDataLoader(const char *ToolName,
     // information.  This can happen when the compiler host and target have
     // different endianness.
     bool ShouldByteSwap = (char)PacketType == 0;
-    PacketType = ShouldByteSwap ? ByteSwap(PacketType) : PacketType;
+    PacketType = ShouldByteSwap ? ByteSwap_32(PacketType) : PacketType;
 
     switch (PacketType) {
       case ArgumentInfo: