[Lldb-commits] [PATCH] D133546: [lldb][fuzz] Allow expression fuzzer to be passed as a flag.

Tue Sep 13 12:40:51 PDT 2022

rupprecht marked an inline comment as done.
rupprecht added inline comments.

================
Comment at: lldb/tools/lldb-fuzzer/lldb-expression-fuzzer/lldb-expression-fuzzer.cpp:66
     ReportError(
         "no target path specified in with the LLDB_FUZZER_TARGET variable");

----------------
cassanova wrote:
> I think having a flag for the fuzzer target alongside the env variable is a good addition. My only wonder is if there should be an error message if no flag is specified so that anyone that uses this knows that they can use a flag as well.
That makes sense -- let me know if the updated wording looks OK.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133546/new/

https://reviews.llvm.org/D133546