[llvm] r229315 - [ADCE] Fix capitalization of some local variables

Hal Finkel hfinkel at anl.gov
Sun Feb 15 07:45:29 PST 2015


Author: hfinkel
Date: Sun Feb 15 09:45:28 2015
New Revision: 229315

URL: http://llvm.org/viewvc/llvm-project?rev=229315&view=rev
Log:
[ADCE] Fix capitalization of some local variables

Bring some local variables in compliance with our naming conventions, NFC.

Modified:
    llvm/trunk/lib/Transforms/Scalar/ADCE.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/ADCE.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/ADCE.cpp?rev=229315&r1=229314&r2=229315&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/ADCE.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/ADCE.cpp Sun Feb 15 09:45:28 2015
@@ -54,8 +54,8 @@ bool ADCE::runOnFunction(Function& F) {
   if (skipOptnoneFunction(F))
     return false;
 
-  SmallPtrSet<Instruction*, 128> alive;
-  SmallVector<Instruction*, 128> worklist;
+  SmallPtrSet<Instruction*, 128> Alive;
+  SmallVector<Instruction*, 128> Worklist;
 
   // Collect the set of "root" instructions that are known live.
   for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I)
@@ -63,37 +63,37 @@ bool ADCE::runOnFunction(Function& F) {
         isa<DbgInfoIntrinsic>(I.getInstructionIterator()) ||
         isa<LandingPadInst>(I.getInstructionIterator()) ||
         I->mayHaveSideEffects()) {
-      alive.insert(I.getInstructionIterator());
-      worklist.push_back(I.getInstructionIterator());
+      Alive.insert(I.getInstructionIterator());
+      Worklist.push_back(I.getInstructionIterator());
     }
 
   // Propagate liveness backwards to operands.
-  while (!worklist.empty()) {
-    Instruction* curr = worklist.pop_back_val();
+  while (!Worklist.empty()) {
+    Instruction* curr = Worklist.pop_back_val();
     for (Instruction::op_iterator OI = curr->op_begin(), OE = curr->op_end();
          OI != OE; ++OI)
       if (Instruction* Inst = dyn_cast<Instruction>(OI))
-        if (alive.insert(Inst).second)
-          worklist.push_back(Inst);
+        if (Alive.insert(Inst).second)
+          Worklist.push_back(Inst);
   }
 
   // The inverse of the live set is the dead set.  These are those instructions
   // which have no side effects and do not influence the control flow or return
   // value of the function, and may therefore be deleted safely.
-  // NOTE: We reuse the worklist vector here for memory efficiency.
+  // NOTE: We reuse the Worklist vector here for memory efficiency.
   for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I)
-    if (!alive.count(I.getInstructionIterator())) {
-      worklist.push_back(I.getInstructionIterator());
+    if (!Alive.count(I.getInstructionIterator())) {
+      Worklist.push_back(I.getInstructionIterator());
       I->dropAllReferences();
     }
 
-  for (SmallVectorImpl<Instruction *>::iterator I = worklist.begin(),
-       E = worklist.end(); I != E; ++I) {
+  for (SmallVectorImpl<Instruction *>::iterator I = Worklist.begin(),
+       E = Worklist.end(); I != E; ++I) {
     ++NumRemoved;
     (*I)->eraseFromParent();
   }
 
-  return !worklist.empty();
+  return !Worklist.empty();
 }
 
 FunctionPass *llvm::createAggressiveDCEPass() {





More information about the llvm-commits mailing list