dawn added a comment. In http://reviews.llvm.org/D12115#226962, @zturner wrote:
> Rather than all of this ugly error-prone code, can we instead use > llvm::sys::fs::root_name to check whether the path contains a drive letter? Sadly no, because root_name only parses c:\ correctly if the platform is Windows. Our IDE is hosted on Windows but talks to lldb running on OSX. I've had this path problem with the DWARF reader and FileSpec - lldb has a core assumption that you only want to support Windows paths if you run on Windows. At least with FileSpec you can pass a PathSyntax, but you have to choose either Windows or Posix, not both. We want to support both Posix and Windows style paths. I've not come up with a clean patch for this problem so none have been submitted. We should start another thread about how to resolve this problem... For now, would it be OK to put a FIXME comment in the source? Repository: rL LLVM http://reviews.llvm.org/D12115 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
