Unit tests would be nice<br><br><div>On Tue Dec 31 2013 at 9:15:41 AM, Maciej Piechotka <<a href="mailto:uzytkownik2@gmail.com">uzytkownik2@gmail.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, 2013-12-31 at 15:30 +0100, Maciej Piechotka wrote:<br>
> Hi,<br>
><br>
> Sorry for double posting - it looks like I run into problems with my MUA<br>
> and I believe my previous message just disappear.<br>
><br>
> It looks like currently it is hard to add value to llvm::StringMap which<br>
> does not have a default constructor (i.e. user needs to allocate<br>
> llvm::StringMapEntry by hand, construct it and fill it effectivly<br>
> reimplementing llvm::StringMapEntry::Create) such as llvm::Regex and it<br>
> is not possible to construct not-copy constructable value without<br>
> default constructor in C++11 by the 'hard' way.<br>
><br>
> The attached patch allows to just copy the value when lvalue reference<br>
> is passed to llvm::StringMapEntry::Create. I thought about adding the<br>
> method for C++11:<br>
><br>
>     template<typename... Args><br>
>     MapEntryTy &GetOrCreateValue(StringRef Key, Args &&... Val)<br>
><br>
> But I'm not sure how it interacted with llvm::<u></u>StringMapEntryInitializer<br>
> use case (there seems to be no use of it in current llvm codebase except<br>
> test and I'm not sure what's the usecase for it).<br>
><br>
> Best regards<br>
<br>
Updated patch - I wasn't aware of LLVM feature macros.<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>