[llvm-commits] [llvm] r52217 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/extractvalue.ll

[Dan Gohman]

On Mon, June 16, 2008 6:08 am, Matthijs Kooijman wrote:
>> It's not permitted to use auto_ptr with arrays, since it uses delete and
>> not delete[]. Can you change this code to use a SmartVector instead?
> Hmm, didn't know that. What exactly do you mean by SmartVector? Just
> std::vector? Does that guarantee that elements are consecutive? Or perhaps
> SmallVector (I think those will have consecutive elements by
> definition...)

Oops, I meant SmallVector. Both SmallVector and std::vector guarantee
their elements are in contiguous memory.

Dan