[Lldb-commits] [PATCH] D31086: Remove FileSystem::MakeDirectory

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 17 12:54:12 PDT 2017


labath added a comment.

In https://reviews.llvm.org/D31086#704122, @zturner wrote:

> That one is calling a file static function `MakeDirectory`, not `FileSystem::MakeDirectory`, and the implementation of that function already calls `llvm::sys::fs::create_directories()` to create the whole tree, so it should be fine.


Right, sorry about that. I was looking at an old checkout. I'm not sure about the PlatformDarwin stuff, but the rest seems fine.


https://reviews.llvm.org/D31086





More information about the lldb-commits mailing list