[llvm-commits] [llvm] r92815 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineCasts.cpp test/Transforms/InstCombine/cast-sext-zext.ll test/Transforms/InstCombine/cast.ll

[Duncan Sands]

Hi Chris,

> +/// If CastOpc is zext, we are asking if the low bits of the value can bit
> +/// computed in a larger type, which is then and'd to get the final result.

bit computed -> be computed

> +/// 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.

I failed to parse this sentence!

> +    // result will be sign-extendy enough to not require these shifts, we can

sign-extendy -> sign-extended

> +    DEBUG(errs() << "ICE: EvaluateInDifferentType converting expression type"

errs() -> dbgs()

Ciao,

Duncan.