[Lldb-commits] [lldb] [lldb][windows] allow long paths in process launcher (PR #206029)

[David Spickett via lldb-commits]

================
@@ -226,16 +227,20 @@ ProcessLauncherWindows::LaunchProcess(const ProcessLaunchInfo &launch_info,
   // command line is not empty, its contents may be modified by CreateProcessW.
   WCHAR *pwcommandLine = wcommandLine.empty() ? nullptr : &wcommandLine[0];
 
-  std::wstring wexecutable, wworkingDirectory;
-  llvm::ConvertUTF8toWide(launch_info.GetExecutableFile().GetPath(),
----------------
DavidSpickett wrote:

Feels like we were already converting the path, but using a function that was less specific.

I support using the specific function, but am I missing a difference between them?

https://github.com/llvm/llvm-project/pull/206029