[Lldb-commits] [PATCH] D17107: [lldb] Unicode support on Win32

[Jim Ingham via lldb-commits]

Unless something has changed since last I looked, the llvm path utilities are host specific.  I didn't inspect the code where this is being used, but it is in Host/common, so unless there's an ifdef WINDOWS around it, I don't think you should use the llvm path utilities.

Jim

> On Feb 10, 2016, at 4:31 PM, Zachary Turner via lldb-commits <lldb-commits at lists.llvm.org> wrote:
> 
> sprintf'ing parent and child paths together to concatenate them doesn't seem like the best idea.  Can we use `llvm::sys::path::append()` here instead?
> 
>