[llvm-commits] [llvm] r133412 - in /llvm/trunk: include/llvm/Constants.h lib/Analysis/ConstantFolding.cpp lib/AsmParser/LLParser.cpp lib/Bitcode/Reader/BitcodeReader.cpp lib/CodeGen/ShadowStackGC.cpp lib/Transforms/IPO/GlobalOpt.cpp lib/Transform

[Jay Foad]

On 21 June 2011 19:40, Chris Lattner <sabre at nondot.org> wrote:
>
> On Jun 21, 2011, at 12:22 AM, Jay Foad wrote:
>
>>>> Patch 1 extends ConstantUniqueMap with a new template parameter
>>>> ValRefType, representing a const reference to ValType. Normally this
>>>> would just be const ValType&, but when ValType is a std::vector, we
>>>> want to use ArrayRef as the reference type.
>>
>>> Wow, looks great to me Jay
>>
>> Just to double check, are you OK with me adding the conversion operator
>>
>>  ArrayRef<T>::operator std::vector<T>() const;
>
> I'm ok with it if there is no no other way

There's always another way; but I don't know what it is, so I've
committed my patch as-is.

Thanks,
Jay.