<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 6, 2016 at 4:44 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="gmail-">On Mon, Jun 6, 2016 at 4:28 PM, Reid Kleckner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">-      BinaryForPath.emplace(Path, OwningBinary<Binary>());<br>
+      OwningBinary<Binary> NullBinary;<br>
+      BinaryForPath.insert(std::make_pair(Path, std::move(NullBinary)));<br></blockquote><div><br></div></span><div>I'm curious - why the introduction of the named variable (NullBinary)? did that work around some particular problem?</div></div></div></div></blockquote><div><br></div><div>I was afraid of std::make_pair using a value type which would be copied, but apparently it works. r<span style="color:rgb(0,0,0)">271964</span></div></div></div></div>