Address review comments.

Change the type used of a parameter.

Author: abidh

llvm/include/llvm/Transforms/Utils/CodeExtractor.h

@@ -291,7 +291,7 @@ class CodeExtractorAnalysisCache {
                           const ValueSet &StructValues, Function *newFunction,
                           StructType *StructArgTy, BasicBlock *header,
                           const ValueSet &SinkingCands,
-                          SmallVector<Value *> &NewValues);
+                          SmallVectorImpl<Value *> &NewValues);
 
     /// Generates a Basic Block that calls the extracted function.
     CallInst *emitReplacerCall(const ValueSet &inputs, const ValueSet &outputs,

llvm/lib/Transforms/Utils/CodeExtractor.cpp

@@ -1630,7 +1630,7 @@ void CodeExtractor::emitFunctionBody(
     const ValueSet &inputs, const ValueSet &outputs,
     const ValueSet &StructValues, Function *newFunction,
     StructType *StructArgTy, BasicBlock *header, const ValueSet &SinkingCands,
-    SmallVector<Value *> &NewValues) {
+    SmallVectorImpl<Value *> &NewValues) {
   Function *oldFunction = header->getParent();
   LLVMContext &Context = oldFunction->getContext();