[PATCH] D36828: Updated a usage of createTemporaryFile that does not expect file to be created.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 17 05:45:12 PDT 2017


ilya-biryukov created this revision.

This fixes a usage of createTemporaryFile in clang repo after
a change in llvm repo.


https://reviews.llvm.org/D36828

Files:
  unittests/Tooling/ToolingTest.cpp


Index: unittests/Tooling/ToolingTest.cpp
===================================================================
--- unittests/Tooling/ToolingTest.cpp
+++ unittests/Tooling/ToolingTest.cpp
@@ -319,8 +319,8 @@
 
 TEST(runToolOnCodeWithArgs, TestNoDepFile) {
   llvm::SmallString<32> DepFilePath;
-  ASSERT_FALSE(
-      llvm::sys::fs::createTemporaryFile("depfile", "d", DepFilePath));
+  ASSERT_FALSE(llvm::sys::fs::getPotentiallyUniqueTempFileName("depfile", "d",
+                                                               DepFilePath));
   std::vector<std::string> Args;
   Args.push_back("-MMD");
   Args.push_back("-MT");


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36828.111506.patch
Type: text/x-patch
Size: 627 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170817/3343f1d3/attachment.bin>


More information about the cfe-commits mailing list