Fix a typo that several people pointed out. Also, address the case of

wrapping that Duncan pointed out.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@94547 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman 2010-01-26 15:56:18 +00:00
parent 058b3f5114
commit 8dcc58e6b4

View File

@ -540,8 +540,10 @@ ConstantRange::add(const ConstantRange &Other) const {
ConstantRange ConstantRange
ConstantRange::multiply(const ConstantRange &Other) const { ConstantRange::multiply(const ConstantRange &Other) const {
// TODO: If either operand is a single element, round the result min anx // TODO: If either operand is a single element and the multiply is known to
// max value to the appropriate multiple of that element. // be non-wrapping, round the result min and max value to the appropriate
// multiple of that element. If wrapping is possible, at least adjust the
// range according to the greatest power-of-two factor of the single element.
if (isEmptySet() || Other.isEmptySet()) if (isEmptySet() || Other.isEmptySet())
return ConstantRange(getBitWidth(), /*isFullSet=*/false); return ConstantRange(getBitWidth(), /*isFullSet=*/false);