[llvm-commits] CVS: llvm/lib/Transforms/ExprTypeConvert.cpp

[Reid Spencer]

Changes in directory llvm/lib/Transforms:

ExprTypeConvert.cpp updated: 1.110 -> 1.111
---
Log message:

For PR786: http://llvm.org/PR786 :
Turn on -Wunused and -Wno-unused-parameter. Clean up most of the resulting
fall out by removing unused variables. Remaining warnings have to do with
unused functions (I didn't want to delete code without review) and unused
variables in generated code. Maintainers should clean up the remaining 
issues when they see them. All changes pass DejaGnu tests and Olden.


---
Diffs of the changes:  (+1 -4)

 ExprTypeConvert.cpp |    5 +----
 1 files changed, 1 insertion(+), 4 deletions(-)


Index: llvm/lib/Transforms/ExprTypeConvert.cpp
diff -u llvm/lib/Transforms/ExprTypeConvert.cpp:1.110 llvm/lib/Transforms/ExprTypeConvert.cpp:1.111
--- llvm/lib/Transforms/ExprTypeConvert.cpp:1.110	Fri May 12 18:32:01 2006
+++ llvm/lib/Transforms/ExprTypeConvert.cpp	Thu Nov  2 14:25:50 2006
@@ -537,7 +537,7 @@
         // a whole structure at a time), so the level raiser must be trying to
         // store into the first field.  Check for this and allow it now:
         //
-        if (const StructType *SElTy = dyn_cast<StructType>(ElTy)) {
+        if (isa<StructType>(ElTy)) {
           unsigned Offset = 0;
           std::vector<Value*> Indices;
           ElTy = getStructOffsetType(ElTy, Offset, Indices, TD, false);
@@ -799,9 +799,6 @@
         Value *SrcPtr = VMCI->second;
 
         if (ElTy != NewTy) {
-          // We check that this is a struct in the initial scan...
-          const StructType *SElTy = cast<StructType>(ElTy);
-
           std::vector<Value*> Indices;
           Indices.push_back(Constant::getNullValue(Type::UIntTy));