mib marked 9 inline comments as done. mib added inline comments.
================ Comment at: lldb/source/Interpreter/CommandInterpreter.cpp:2929 + Status error; + user_id_t fd_dst = FileCache::GetInstance().OpenFile( + FileSpec(output_file), flags, lldb::eFilePermissionsFileDefault, error); ---------------- JDevlieghere wrote: > Why are you going through the `FileCache`? I was thinking if the user saves several times during his session to the same file, that might be useful. Looking at the implementation, it uses the FileSystem instance, so I'm not sure why that would a problem ... May be you could elaborate why I shouldn't use it ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82155/new/ https://reviews.llvm.org/D82155 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
