diff options
author | Chris Lattner <sabre@nondot.org> | 2010-01-06 05:32:15 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2010-01-06 05:32:15 +0000 |
commit | 5c24a6da29784a0b08cf4b1184e780b295951862 (patch) | |
tree | e17ee927fba98e52d9ab94544f42afc989bdcd51 | |
parent | b7339d079d410524f59066b2987fb91f5193dd32 (diff) | |
download | external_llvm-5c24a6da29784a0b08cf4b1184e780b295951862.zip external_llvm-5c24a6da29784a0b08cf4b1184e780b295951862.tar.gz external_llvm-5c24a6da29784a0b08cf4b1184e780b295951862.tar.bz2 |
tweaks suggested by Duncan
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@92824 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/InstCombine/InstCombineCasts.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/lib/Transforms/InstCombine/InstCombineCasts.cpp b/lib/Transforms/InstCombine/InstCombineCasts.cpp index acd78d6..d49c5a1 100644 --- a/lib/Transforms/InstCombine/InstCombineCasts.cpp +++ b/lib/Transforms/InstCombine/InstCombineCasts.cpp @@ -162,7 +162,7 @@ Instruction *InstCombiner::PromoteCastOfAllocation(BitCastInst &CI, /// inst(trunc(x),trunc(y)), which only makes sense if x and y can be /// efficiently truncated. /// -/// If CastOpc is zext, we are asking if the low bits of the value can bit +/// If CastOpc is zext, we are asking if the low bits of the value can be /// computed in a larger type, which is then and'd to get the final result. static bool CanEvaluateInDifferentType(Value *V, const Type *Ty, unsigned CastOpc, @@ -295,9 +295,9 @@ static bool CanEvaluateInDifferentType(Value *V, const Type *Ty, /// CanEvaluateSExtd - Return true if we can take the specified value /// and return it as type Ty without inserting any new casts and without -/// changing the computed value of the common low bits. This is used by code -/// that tries to promote integer operations to a wider types will allow us to -/// eliminate the extension. +/// changing the value of the common low bits. This is used by code that tries +/// to promote integer operations to a wider types will allow us to eliminate +/// the extension. /// /// This returns 0 if we can't do this or the number of sign bits that would be /// set if we can. For example, CanEvaluateSExtd(i16 1, i64) would return 63, @@ -622,14 +622,14 @@ Instruction *InstCombiner::commonIntCastTransforms(CastInst &CI) { // Because this is a sign extension, we can always transform it by inserting // two new shifts (to do the extension). However, this is only profitable // if we've eliminated two or more casts from the input. If we know the - // result will be sign-extendy enough to not require these shifts, we can + // result will be sign-extended enough to not require these shifts, we can // always do the transformation. if (NumCastsRemoved < 2 && NumBitsSExt <= DestBitSize-SrcBitSize) return 0; // Okay, we can transform this! Insert the new expression now. - DEBUG(errs() << "ICE: EvaluateInDifferentType converting expression type" + DEBUG(dbgs() << "ICE: EvaluateInDifferentType converting expression type" " to avoid sign extend: " << CI); Value *Res = EvaluateInDifferentType(Src, DestTy, true); assert(Res->getType() == DestTy); @@ -645,7 +645,7 @@ Instruction *InstCombiner::commonIntCastTransforms(CastInst &CI) { } } - DEBUG(errs() << "ICE: EvaluateInDifferentType converting expression type" + DEBUG(dbgs() << "ICE: EvaluateInDifferentType converting expression type" " to avoid cast: " << CI); Value *Res = EvaluateInDifferentType(Src, DestTy, false); assert(Res->getType() == DestTy); |