[PATCH] [lld]Fix a temporary string use

Ron Ofir ron.ofir at gmail.com
Fri Aug 23 14:50:24 PDT 2013


Hi ruiu, shankarke,

Fixes an additional temporary string use

http://llvm-reviews.chandlerc.com/D1494

Files:
  lib/Driver/WinLinkDriver.cpp

Index: lib/Driver/WinLinkDriver.cpp
===================================================================
--- lib/Driver/WinLinkDriver.cpp
+++ lib/Driver/WinLinkDriver.cpp
@@ -410,7 +410,7 @@
     SmallString<128> firstInputFilePath =
         (llvm::dyn_cast<FileNode>(&((inputGraph)[0])))->path(ctx);
     (llvm::sys::path::replace_extension(firstInputFilePath, ".exe"));
-    ctx.setOutputPath(firstInputFilePath.str());
+    ctx.setOutputPath(ctx.allocateString(firstInputFilePath.str()));
   }
 
   // Validate the combination of options used.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1494.1.patch
Type: text/x-patch
Size: 547 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130823/4bf8289c/attachment.bin>


More information about the llvm-commits mailing list