[llvm] r328018 - [llvm-mca] Remove const from a bunch of ArrayRef. NFC

Andrea Di Biagio via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 20 12:06:35 PDT 2018


Author: adibiagio
Date: Tue Mar 20 12:06:34 2018
New Revision: 328018

URL: http://llvm.org/viewvc/llvm-project?rev=328018&view=rev
Log:
[llvm-mca] Remove const from a bunch of ArrayRef. NFC

Modified:
    llvm/trunk/tools/llvm-mca/Dispatch.cpp
    llvm/trunk/tools/llvm-mca/Dispatch.h
    llvm/trunk/tools/llvm-mca/HWEventListener.h
    llvm/trunk/tools/llvm-mca/InstrBuilder.cpp
    llvm/trunk/tools/llvm-mca/Scheduler.cpp
    llvm/trunk/tools/llvm-mca/Scheduler.h

Modified: llvm/trunk/tools/llvm-mca/Dispatch.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/Dispatch.cpp?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/Dispatch.cpp (original)
+++ llvm/trunk/tools/llvm-mca/Dispatch.cpp Tue Mar 20 12:06:34 2018
@@ -150,7 +150,7 @@ void RegisterFile::collectWrites(SmallVe
   }
 }
 
-unsigned RegisterFile::isAvailable(const ArrayRef<unsigned> Regs) const {
+unsigned RegisterFile::isAvailable(ArrayRef<unsigned> Regs) const {
   SmallVector<unsigned, 4> NumTemporaries(getNumRegisterFiles());
 
   // Find how many new mappings must be created for each register file.

Modified: llvm/trunk/tools/llvm-mca/Dispatch.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/Dispatch.h?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/Dispatch.h (original)
+++ llvm/trunk/tools/llvm-mca/Dispatch.h Tue Mar 20 12:06:34 2018
@@ -134,7 +134,7 @@ public:
   // For example: if all register files are available, then the response mask
   // is a bitmask of all zeroes. If Instead register file #1 is not available,
   // then the response mask is 0b10.
-  unsigned isAvailable(const llvm::ArrayRef<unsigned> Regs) const;
+  unsigned isAvailable(llvm::ArrayRef<unsigned> Regs) const;
   void collectWrites(llvm::SmallVectorImpl<WriteState *> &Writes,
                      unsigned RegID) const;
   void updateOnRead(ReadState &RS, unsigned RegID);

Modified: llvm/trunk/tools/llvm-mca/HWEventListener.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/HWEventListener.h?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/HWEventListener.h (original)
+++ llvm/trunk/tools/llvm-mca/HWEventListener.h Tue Mar 20 12:06:34 2018
@@ -48,8 +48,8 @@ public:
     LastGenericEventType,
   };
 
-  HWInstructionEvent(unsigned Type, unsigned Index)
-      : Type(Type), Index(Index) {}
+  HWInstructionEvent(unsigned type, unsigned index)
+      : Type(type), Index(index) {}
 
   // The event type. The exact meaning depends on the subtarget.
   const unsigned Type;
@@ -61,11 +61,11 @@ class HWInstructionIssuedEvent : public
 public:
   using ResourceRef = std::pair<uint64_t, uint64_t>;
   HWInstructionIssuedEvent(
-      unsigned Index, const llvm::ArrayRef<std::pair<ResourceRef, unsigned>> UR)
+      unsigned Index, llvm::ArrayRef<std::pair<ResourceRef, unsigned>> UR)
       : HWInstructionEvent(HWInstructionEvent::Issued, Index),
         UsedResources(UR) {}
 
-  const llvm::ArrayRef<std::pair<ResourceRef, unsigned>> UsedResources;
+  llvm::ArrayRef<std::pair<ResourceRef, unsigned>> UsedResources;
 };
 
 // A HWStallEvent represents a pipeline stall caused by the lack of hardware

Modified: llvm/trunk/tools/llvm-mca/InstrBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/InstrBuilder.cpp?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/InstrBuilder.cpp (original)
+++ llvm/trunk/tools/llvm-mca/InstrBuilder.cpp Tue Mar 20 12:06:34 2018
@@ -26,7 +26,7 @@ using namespace llvm;
 static void
 initializeUsedResources(InstrDesc &ID, const MCSchedClassDesc &SCDesc,
                         const MCSubtargetInfo &STI,
-                        const ArrayRef<uint64_t> ProcResourceMasks) {
+                        ArrayRef<uint64_t> ProcResourceMasks) {
   const MCSchedModel &SM = STI.getSchedModel();
 
   // Populate resources consumed.

Modified: llvm/trunk/tools/llvm-mca/Scheduler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/Scheduler.cpp?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/Scheduler.cpp (original)
+++ llvm/trunk/tools/llvm-mca/Scheduler.cpp Tue Mar 20 12:06:34 2018
@@ -124,7 +124,7 @@ void ResourceManager::release(ResourceRe
 }
 
 ResourceStateEvent
-ResourceManager::canBeDispatched(const ArrayRef<uint64_t> Buffers) const {
+ResourceManager::canBeDispatched(ArrayRef<uint64_t> Buffers) const {
   ResourceStateEvent Result = ResourceStateEvent::RS_BUFFER_AVAILABLE;
   for (uint64_t Buffer : Buffers) {
     Result = isBufferAvailable(Buffer);
@@ -134,7 +134,7 @@ ResourceManager::canBeDispatched(const A
   return Result;
 }
 
-void ResourceManager::reserveBuffers(const ArrayRef<uint64_t> Buffers) {
+void ResourceManager::reserveBuffers(ArrayRef<uint64_t> Buffers) {
   for (const uint64_t R : Buffers) {
     reserveBuffer(R);
     ResourceState &Resource = *Resources[R];
@@ -145,7 +145,7 @@ void ResourceManager::reserveBuffers(con
   }
 }
 
-void ResourceManager::releaseBuffers(const ArrayRef<uint64_t> Buffers) {
+void ResourceManager::releaseBuffers(ArrayRef<uint64_t> Buffers) {
   for (const uint64_t R : Buffers)
     releaseBuffer(R);
 }
@@ -423,7 +423,7 @@ void Scheduler::updateIssuedQueue() {
 }
 
 void Scheduler::notifyInstructionIssued(
-    unsigned Index, const ArrayRef<std::pair<ResourceRef, unsigned>> &Used) {
+    unsigned Index, ArrayRef<std::pair<ResourceRef, unsigned>> Used) {
   DEBUG({
     dbgs() << "[E] Instruction Issued: " << Index << '\n';
     for (const std::pair<ResourceRef, unsigned> &Resource : Used) {

Modified: llvm/trunk/tools/llvm-mca/Scheduler.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mca/Scheduler.h?rev=328018&r1=328017&r2=328018&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-mca/Scheduler.h (original)
+++ llvm/trunk/tools/llvm-mca/Scheduler.h Tue Mar 20 12:06:34 2018
@@ -339,10 +339,10 @@ public:
 
   // Consume a slot in every buffered resource from array 'Buffers'. Resource
   // units that are dispatch hazards (i.e. BufferSize=0) are marked as reserved.
-  void reserveBuffers(const llvm::ArrayRef<uint64_t> Buffers);
+  void reserveBuffers(llvm::ArrayRef<uint64_t> Buffers);
 
   // Release buffer entries previously allocated by method reserveBuffers.
-  void releaseBuffers(const llvm::ArrayRef<uint64_t> Buffers);
+  void releaseBuffers(llvm::ArrayRef<uint64_t> Buffers);
 
   void reserveResource(uint64_t ResourceID) {
     ResourceState &Resource = *Resources[ResourceID];
@@ -420,7 +420,7 @@ class Scheduler {
 
   void notifyInstructionIssued(
       unsigned Index,
-      const llvm::ArrayRef<std::pair<ResourceRef, unsigned>> &Used);
+      llvm::ArrayRef<std::pair<ResourceRef, unsigned>> Used);
   void notifyInstructionExecuted(unsigned Index);
   void notifyInstructionReady(unsigned Index);
   void notifyResourceAvailable(const ResourceRef &RR);




More information about the llvm-commits mailing list