[PATCH] D60630: [Support] Add a test for recursive response file expansion

[Hans Wennborg via Phabricator via llvm-commits]

hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.

Looks good to me.



================
Comment at: llvm/unittests/Support/CommandLineTest.cpp:730
+  sys::path::append(ResponseFilePath, TestDir, "recursive.rsp");
+  std::string ResponseFileRef = std::string("@") + ResponseFilePath.c_str();
+
----------------
Not that it matters, but would using .str() instead of .c_str() work (relying on StringRef's conversion operator to std::sttring)?


================
Comment at: llvm/unittests/Support/CommandLineTest.cpp:732
+
+  std::ofstream ResponseFile(ResponseFilePath.c_str());
+  EXPECT_TRUE(ResponseFile.is_open());
----------------
ditto


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60630/new/

https://reviews.llvm.org/D60630