diff options
author | Jordan Rose <jordan_rose@apple.com> | 2013-05-09 21:07:43 +0000 |
---|---|---|
committer | Jordan Rose <jordan_rose@apple.com> | 2013-05-09 21:07:43 +0000 |
commit | d044f65e37f09a6c69da4d002c1e8957bd4d99c0 (patch) | |
tree | 000bf6cc3cf64d6d34f7197da326f25fe49977b7 /include | |
parent | edfef3bd27d6269d473fbc570e8c2be02b4070df (diff) | |
download | llvm-d044f65e37f09a6c69da4d002c1e8957bd4d99c0.tar.gz llvm-d044f65e37f09a6c69da4d002c1e8957bd4d99c0.tar.bz2 llvm-d044f65e37f09a6c69da4d002c1e8957bd4d99c0.tar.xz |
Rewrite assert to avoid warning when the record element type is byte-sized.
BitstreamWriter asserts that when blob data is written from the record
element vector, each element fits in a byte. However, if the record
elements are specified as a SmallVector of 'char', this causes a warning
from -Wtautological-constant-out-of-range-compare. Fix this by using
llvm::isUInt<8> instead of a plain comparison against 256.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@181545 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include')
-rw-r--r-- | include/llvm/Bitcode/BitstreamWriter.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/llvm/Bitcode/BitstreamWriter.h b/include/llvm/Bitcode/BitstreamWriter.h index a837211875..f40a0d1d25 100644 --- a/include/llvm/Bitcode/BitstreamWriter.h +++ b/include/llvm/Bitcode/BitstreamWriter.h @@ -381,7 +381,8 @@ private: BlobData = 0; } else { for (unsigned e = Vals.size(); RecordIdx != e; ++RecordIdx) { - assert(Vals[RecordIdx] < 256 && "Value too large to emit as blob"); + assert(isUInt<8>(Vals[RecordIdx]) && + "Value too large to emit as blob"); WriteByte((unsigned char)Vals[RecordIdx]); } } |