From e6ac9f5ec3b890885abef8c4a315203ef02a25e5 Mon Sep 17 00:00:00 2001 From: Rui Ueyama Date: Tue, 17 Apr 2018 21:09:16 +0000 Subject: Rename sys::Process::GetArgumentVector -> sys::windows::GetCommandLineArguments GetArgumentVector (or GetCommandLineArguments) is very Windows-specific. I think it doesn't make much sense to provide that function from sys::Process. I also made a change so that the function takes a BumpPtrAllocator instead of a SpecificBumpPtrAllocator. The latter is the class to call dtors, but since char * is trivially destructible, we should use the former class. Differential Revision: https://reviews.llvm.org/D45641 llvm-svn: 330216 --- llvm/lib/Support/Unix/Process.inc | 9 --------- 1 file changed, 9 deletions(-) (limited to 'llvm/lib/Support/Unix/Process.inc') diff --git a/llvm/lib/Support/Unix/Process.inc b/llvm/lib/Support/Unix/Process.inc index 7a4e3861400..3ce2212c6e7 100644 --- a/llvm/lib/Support/Unix/Process.inc +++ b/llvm/lib/Support/Unix/Process.inc @@ -172,15 +172,6 @@ Optional Process::GetEnv(StringRef Name) { return std::string(Val); } -std::error_code -Process::GetArgumentVector(SmallVectorImpl &ArgsOut, - ArrayRef ArgsIn, - SpecificBumpPtrAllocator &) { - ArgsOut.append(ArgsIn.begin(), ArgsIn.end()); - - return std::error_code(); -} - namespace { class FDCloser { public: -- cgit v1.2.3