[PATCH] D16063: [Analyzer] Use a wider integer type for an array index
Gábor Horváth via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 11 08:10:08 PST 2016
xazax.hun added a comment.
In http://reviews.llvm.org/D16063#323513, @a.sidorin wrote:
> As I described before, PtrDiffType is signed and is limited to SIZE_MAX/2. However, we are allowed to create arrays with the size more than SIZE_MAX/2 (see testIndexTooBig() test for details). But we should not loose the ability to handle such arrays and their indices.That's why I selected a 64-bit arch-independent type.
Oh, I see now, thanks. My only concern left is that, is it guaranteed that LongLongTy is available on all supported platforms including embedded ones?
Repository:
rL LLVM
http://reviews.llvm.org/D16063
More information about the cfe-commits
mailing list