JDevlieghere added inline comments.
================ Comment at: lldb/tools/lldb-fuzzer/CMakeLists.txt:1-2 add_subdirectory(lldb-target-fuzzer) +add_subdirectory(lldb-commandinterpreter-fuzzer) add_subdirectory(utils) ---------------- nit: I'd sort these alphabetically ================ Comment at: lldb/tools/lldb-fuzzer/lldb-commandinterpreter-fuzzer/CMakeLists.txt:15 + liblldb + lldbFuzzerUtils + ) ---------------- I don't think this used any longer. ================ Comment at: lldb/tools/lldb-fuzzer/lldb-commandinterpreter-fuzzer/lldb-commandinterpreter-fuzzer.cpp:1-2 +//===-- lldb-commandinterpreter-fuzzer.cpp - Fuzz LLDB's command interpreter +//---------------------===// +// ---------------- ASCII art needs fixing :-) ================ Comment at: lldb/tools/lldb-fuzzer/lldb-commandinterpreter-fuzzer/lldb-commandinterpreter-fuzzer.cpp:17 +#include "lldb/API/SBTarget.h" +#include "utils/TempFile.h" + ---------------- Not used? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128292/new/ https://reviews.llvm.org/D128292 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
