[llvm] r330489 - [ObjCARC] Take BlockColors by const reference. NFC

Shoaib Meenai via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 20 15:14:45 PDT 2018


Author: smeenai
Date: Fri Apr 20 15:14:45 2018
New Revision: 330489

URL: http://llvm.org/viewvc/llvm-project?rev=330489&view=rev
Log:
[ObjCARC] Take BlockColors by const reference. NFC

Modified:
    llvm/trunk/lib/Transforms/ObjCARC/ObjCARCContract.cpp
    llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp

Modified: llvm/trunk/lib/Transforms/ObjCARC/ObjCARCContract.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/ObjCARC/ObjCARCContract.cpp?rev=330489&r1=330488&r2=330489&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/ObjCARC/ObjCARCContract.cpp (original)
+++ llvm/trunk/lib/Transforms/ObjCARC/ObjCARCContract.cpp Fri Apr 20 15:14:45 2018
@@ -75,13 +75,12 @@ namespace {
     SmallPtrSet<CallInst *, 8> StoreStrongCalls;
 
     /// Returns true if we eliminated Inst.
-    bool
-    tryToPeepholeInstruction(Function &F, Instruction *Inst,
-                             inst_iterator &Iter,
-                             SmallPtrSetImpl<Instruction *> &DepInsts,
-                             SmallPtrSetImpl<const BasicBlock *> &Visited,
-                             bool &TailOkForStoreStrong,
-                             DenseMap<BasicBlock *, ColorVector> &BlockColors);
+    bool tryToPeepholeInstruction(
+        Function &F, Instruction *Inst, inst_iterator &Iter,
+        SmallPtrSetImpl<Instruction *> &DepInsts,
+        SmallPtrSetImpl<const BasicBlock *> &Visited,
+        bool &TailOkForStoreStrong,
+        const DenseMap<BasicBlock *, ColorVector> &BlockColors);
 
     bool optimizeRetainCall(Function &F, Instruction *Retain);
 
@@ -431,7 +430,7 @@ bool ObjCARCContract::tryToPeepholeInstr
   SmallPtrSetImpl<Instruction *> &DependingInsts,
   SmallPtrSetImpl<const BasicBlock *> &Visited,
   bool &TailOkForStoreStrongs,
-  DenseMap<BasicBlock *, ColorVector> &BlockColors) {
+  const DenseMap<BasicBlock *, ColorVector> &BlockColors) {
     // Only these library routines return their argument. In particular,
     // objc_retainBlock does not necessarily return its argument.
   ARCInstKind Class = GetBasicARCInstKind(Inst);

Modified: llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp?rev=330489&r1=330488&r2=330489&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp (original)
+++ llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp Fri Apr 20 15:14:45 2018
@@ -688,7 +688,7 @@ void ObjCARCOpt::OptimizeAutoreleaseRVCa
 namespace {
 Instruction *
 CloneCallInstForBB(CallInst &CI, BasicBlock &BB,
-                   DenseMap<BasicBlock *, ColorVector> &BlockColors) {
+                   const DenseMap<BasicBlock *, ColorVector> &BlockColors) {
   SmallVector<OperandBundleDef, 1> OpBundles;
   for (unsigned I = 0, E = CI.getNumOperandBundles(); I != E; ++I) {
     auto Bundle = CI.getOperandBundleAt(I);




More information about the llvm-commits mailing list