<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 23, 2013 at 2:50 PM, Ron Ofir <span dir="ltr"><<a href="mailto:ron.ofir@gmail.com" target="_blank">ron.ofir@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi ruiu, shankarke,<br>
<br>
Fixes an additional temporary string use<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1494" target="_blank">http://llvm-reviews.chandlerc.com/D1494</a><br>
<br>
Files:<br>
  lib/Driver/WinLinkDriver.cpp<br>
<br>
Index: lib/Driver/WinLinkDriver.cpp<br>
===================================================================<br>
--- lib/Driver/WinLinkDriver.cpp<br>
+++ lib/Driver/WinLinkDriver.cpp<br>
@@ -410,7 +410,7 @@<br>
     SmallString<128> firstInputFilePath =<br>
         (llvm::dyn_cast<FileNode>(&((inputGraph)[0])))->path(ctx);<br>
     (llvm::sys::path::replace_extension(firstInputFilePath, ".exe"));<br>
-    ctx.setOutputPath(firstInputFilePath.str());<br>
+    ctx.setOutputPath(ctx.allocateString(firstInputFilePath.str()));<br>
   }<br>
<br>
   // Validate the combination of options used.<br>
</blockquote></div><br></div>