diff options
| author | Dan Gohman <gohman@apple.com> | 2012-09-13 18:19:06 +0000 | 
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2012-09-13 18:19:06 +0000 | 
| commit | d0080c45f90afb7abcf6c4f5733c239422836819 (patch) | |
| tree | ed57766e625365fce68bbee251bc2743e303c1d2 /llvm | |
| parent | e5c0a9bad4023b33fe5f8fcc8bb6a74e5c75caec (diff) | |
| download | bcm5719-llvm-d0080c45f90afb7abcf6c4f5733c239422836819.tar.gz bcm5719-llvm-d0080c45f90afb7abcf6c4f5733c239422836819.zip  | |
Extract code for reducing a type to a single value type into a helper function.
llvm-svn: 163817
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp | 36 | 
1 files changed, 21 insertions, 15 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp index b12fc013576..1b389b92e8d 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp @@ -29,6 +29,26 @@ static Type *getPromotedType(Type *Ty) {    return Ty;  } +/// reduceToSingleValueType - Given an aggregate type which ultimately holds a +/// single scalar element, like {{{type}}} or [1 x type], return type. +static Type *reduceToSingleValueType(Type *T) { +  while (!T->isSingleValueType()) { +    if (StructType *STy = dyn_cast<StructType>(T)) { +      if (STy->getNumElements() == 1) +        T = STy->getElementType(0); +      else +        break; +    } else if (ArrayType *ATy = dyn_cast<ArrayType>(T)) { +      if (ATy->getNumElements() == 1) +        T = ATy->getElementType(); +      else +        break; +    } else +      break; +  } + +  return T; +}  Instruction *InstCombiner::SimplifyMemTransfer(MemIntrinsic *MI) {    unsigned DstAlign = getKnownAlignment(MI->getArgOperand(0), TD); @@ -80,20 +100,7 @@ Instruction *InstCombiner::SimplifyMemTransfer(MemIntrinsic *MI) {      if (TD && SrcETy->isSized() && TD->getTypeStoreSize(SrcETy) == Size) {        // The SrcETy might be something like {{{double}}} or [1 x double].  Rip        // down through these levels if so. -      while (!SrcETy->isSingleValueType()) { -        if (StructType *STy = dyn_cast<StructType>(SrcETy)) { -          if (STy->getNumElements() == 1) -            SrcETy = STy->getElementType(0); -          else -            break; -        } else if (ArrayType *ATy = dyn_cast<ArrayType>(SrcETy)) { -          if (ATy->getNumElements() == 1) -            SrcETy = ATy->getElementType(); -          else -            break; -        } else -          break; -      } +      SrcETy = reduceToSingleValueType(SrcETy);        if (SrcETy->isSingleValueType()) {          NewSrcPtrTy = PointerType::get(SrcETy, SrcAddrSp); @@ -102,7 +109,6 @@ Instruction *InstCombiner::SimplifyMemTransfer(MemIntrinsic *MI) {      }    } -    // If the memcpy/memmove provides better alignment info than we can    // infer, use it.    SrcAlign = std::max(SrcAlign, CopyAlign);  | 

