[llvm] r295500 - [NewGVN] createVariableOrConstant is not required anymore. NFCI.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 17 13:55:47 PST 2017


Author: davide
Date: Fri Feb 17 15:55:47 2017
New Revision: 295500

URL: http://llvm.org/viewvc/llvm-project?rev=295500&view=rev
Log:
[NewGVN] createVariableOrConstant is not required anymore. NFCI.

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

Modified: llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp?rev=295500&r1=295499&r2=295500&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp Fri Feb 17 15:55:47 2017
@@ -308,7 +308,6 @@ private:
   PHIExpression *createPHIExpression(Instruction *);
   const VariableExpression *createVariableExpression(Value *);
   const ConstantExpression *createConstantExpression(Constant *);
-  const Expression *createVariableOrConstant(Value *V);
   const UnknownExpression *createUnknownExpression(Instruction *);
   const StoreExpression *createStoreExpression(StoreInst *, MemoryAccess *);
   LoadExpression *createLoadExpression(Type *, Value *, LoadInst *,
@@ -671,13 +670,6 @@ const VariableExpression *NewGVN::create
   return E;
 }
 
-const Expression *NewGVN::createVariableOrConstant(Value *V) {
-  auto Leader = lookupOperandLeader(V);
-  if (auto *C = dyn_cast<Constant>(Leader))
-    return createConstantExpression(C);
-  return createVariableExpression(Leader);
-}
-
 const ConstantExpression *NewGVN::createConstantExpression(Constant *C) {
   auto *E = new (ExpressionAllocator) ConstantExpression(C);
   E->setOpcode(C->getValueID());




More information about the llvm-commits mailing list