Hi Aaron, + ConstantInt *Length = cast<ConstantInt>(MSI->getLength()); + unsigned NumBytes = Length->getZExtValue(); + if (NumBytes != 0 && !Length->isNegative()) { instead you could do: int NumBytes = Length->getSExtValue(); if (NumBytes > 0) { Ciao, Duncan.