[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