compnerd marked an inline comment as done. compnerd added inline comments.
================ Comment at: lldb/source/Plugins/Platform/Windows/PlatformWindows.h:47-51 + uint32_t DoLoadImage(lldb_private::Process *process, + const lldb_private::FileSpec &remote_file, + const std::vector<std::string> *paths, + lldb_private::Status &error, + lldb_private::FileSpec *loaded_path) override; ---------------- JDevlieghere wrote: > I know it's an `override`, but what a horrible interface... I completely agree. ================ Comment at: lldb/test/Shell/Process/Windows/process_load.cpp:3 + +// REQUIRES: system-windows +// RUN: %build --compiler=clang-cl -o %t.exe -- %s ---------------- JDevlieghere wrote: > compnerd wrote: > > JDevlieghere wrote: > > > We should probably have a `lit.local.cfg` in the Windows directory with > > > > > > ``` > > > if 'system-windows' not in config.available_features: > > > config.unsupported = True > > > > > > ``` > > I think that's a good idea, but, should be a separate change - it isn't > > related to the load/unload functionality. > Fair enough I already pushed that change, so it's already taken care of, but it is not part of the set of changes :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77287/new/ https://reviews.llvm.org/D77287 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
