[llvm] r328340 - [llvm-mca] update the ResourcePressureView after r328335. NFC.

Andrea Di Biagio via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 23 10:53:02 PDT 2018


Author: adibiagio
Date: Fri Mar 23 10:53:02 2018
New Revision: 328340

URL: http://llvm.org/viewvc/llvm-project?rev=328340&view=rev
Log:
[llvm-mca] update the ResourcePressureView after r328335. NFC.

This should have been part of r328335. I forgot to svn add these files.

Modified:
    llvm/trunk/tools/llvm-mca/ResourcePressureView.cpp
    llvm/trunk/tools/llvm-mca/ResourcePressureView.h

Modified: llvm/trunk/tools/llvm-mca/ResourcePressureView.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/ResourcePressureView.cpp?rev=328340&r1=328339&r2=328340&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/ResourcePressureView.cpp (original)
+++ llvm/trunk/tools/llvm-mca/ResourcePressureView.cpp Fri Mar 23 10:53:02 2018
@@ -36,7 +36,7 @@ void ResourcePressureView::initialize()
 
   NumResourceUnits = R2VIndex;
   ResourceUsage.resize(NumResourceUnits * (Source.size() + 1));
-  std::fill(ResourceUsage.begin(), ResourceUsage.end(), 0);
+  std::fill(ResourceUsage.begin(), ResourceUsage.end(), 0.0);
 }
 
 void ResourcePressureView::onInstructionEvent(const HWInstructionEvent &Event) {
@@ -45,7 +45,7 @@ void ResourcePressureView::onInstruction
     return;
   const auto &IssueEvent = static_cast<const HWInstructionIssuedEvent &>(Event);
   unsigned SourceIdx = Event.Index % Source.size();
-  for (const std::pair<ResourceRef, unsigned> &Use : IssueEvent.UsedResources) {
+  for (const std::pair<ResourceRef, double> &Use : IssueEvent.UsedResources) {
     const ResourceRef &RR = Use.first;
     assert(Resource2VecIndex.find(RR.first) != Resource2VecIndex.end());
     unsigned R2VIndex = Resource2VecIndex[RR.first];
@@ -113,13 +113,13 @@ void ResourcePressureView::printResource
   TempStream << '\n';
 
   for (unsigned I = 0, E = NumResourceUnits; I < E; ++I) {
-    unsigned Usage = ResourceUsage[I + Source.size() * E];
+    double Usage = ResourceUsage[I + Source.size() * E];
     if (!Usage) {
       TempStream << " -     ";
       continue;
     }
 
-    double Pressure = (double)Usage / Executions;
+    double Pressure = Usage / Executions;
     TempStream << format("%.2f", Pressure);
     if (Pressure < 10.0)
       TempStream << "   ";
@@ -144,11 +144,11 @@ void ResourcePressureView::printResource
 
   for (unsigned I = 0, E = Source.size(); I < E; ++I) {
     for (unsigned J = 0; J < NumResourceUnits; ++J) {
-      unsigned Usage = ResourceUsage[J + I * NumResourceUnits];
-      if (Usage == 0) {
+      double Usage = ResourceUsage[J + I * NumResourceUnits];
+      if (!Usage) {
         TempStream << " -     ";
       } else {
-        double Pressure = (double)Usage / Executions;
+        double Pressure = Usage / Executions;
         if (Pressure < 0.005) {
           TempStream << " -     ";
         } else {

Modified: llvm/trunk/tools/llvm-mca/ResourcePressureView.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/ResourcePressureView.h?rev=328340&r1=328339&r2=328340&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/ResourcePressureView.h (original)
+++ llvm/trunk/tools/llvm-mca/ResourcePressureView.h Fri Mar 23 10:53:02 2018
@@ -81,7 +81,7 @@ class ResourcePressureView : public View
   llvm::DenseMap<unsigned, unsigned> Resource2VecIndex;
 
   // Table of resources used by instructions.
-  std::vector<unsigned> ResourceUsage;
+  std::vector<double> ResourceUsage;
   unsigned NumResourceUnits;
 
   const llvm::MCInst &GetMCInstFromIndex(unsigned Index) const;




More information about the llvm-commits mailing list