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

[Chelsea Cassanova via Phabricator via lldb-commits]

cassanova 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");
 
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133546