ADT: SmallVector size/capacity use word-size integers when elements are small
authorAndrew Browne <browneee@google.com>
Sat, 25 Apr 2020 02:26:11 +0000 (19:26 -0700)
committerAndrew Browne <browneee@google.com>
Mon, 27 Apr 2020 20:41:01 +0000 (13:41 -0700)
commitdda3c19a3618dce9492687f8e880e7a73486ee98
tree554e6678970b3e1b91e07a59e2fcaa8044a4cf8f
parentf8990feb125a0f8d3f2892a589bc6fad3c430858
ADT: SmallVector size/capacity use word-size integers when elements are small

SmallVector currently uses 32bit integers for size and capacity to reduce
sizeof(SmallVector). This limits the number of elements to UINT32_MAX.

For a SmallVector<char>, this limits the SmallVector size to only 4GB.
Buffering bitcode output uses SmallVector<char>, but needs >4GB output.

This changes SmallVector size and capacity to conditionally use word-size
integers if the element type is small (<4 bytes). For larger elements types,
the vector size can reach ~16GB with 32bit size.

Making this conditional on the element type provides both the smaller
sizeof(SmallVector) for larger types which are unlikely to grow so large,
and supports larger capacities for smaller element types.

This recommit fixes the same template being instantiated twice on platforms
where uintptr_t is the same as uint32_t.
llvm/include/llvm/ADT/SmallVector.h
llvm/lib/Support/SmallVector.cpp