[PATCH] D131020: Reland "[lldb/Fuzzer] Add fuzzer for expression evaluator"

cassanova updated this revision to Diff 449729. cassanova added a comment. Set the LLDB_FUZZER_TARGET environment variable in the CMake file for the source code to use. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131020/new/ https://reviews.llvm.org/D131020 Files: clang/cmake/modu

[PATCH] D131020: Reland "[lldb/Fuzzer] Add fuzzer for expression evaluator"

cassanova created this revision. cassanova added reviewers: JDevlieghere, mib. cassanova added a project: LLDB. Herald added a subscriber: mgorny. Herald added a project: All. cassanova requested review of this revision. Herald added a project: clang. Herald added subscribers: lldb-commits, cfe-com

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

This revision was automatically updated to reflect the committed changes. Closed by commit rGb797834748f1: [lldb/Fuzzer] Add fuzzer for expression evaluator (authored by cassanova). Changed prior to commit: https://reviews.llvm.org/D129377?vs=444761&id=446982#toc Repository: rG LLVM Github M

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova updated this revision to Diff 444761. cassanova edited the summary of this revision. cassanova added a comment. Removed the cxx_proto.proto file since we include the headers that it generates from the clang side. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129377/new/ https

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova marked an inline comment as done. cassanova added inline comments. Comment at: lldb/tools/lldb-fuzzer/lldb-expression-fuzzer/cxx_proto.proto:1 +//===-- cxx_proto.proto - Protobuf description of C++ -===// +// JDevlieghere wrote: > D

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova updated this revision to Diff 444361. cassanova added a comment. Updated the ProtobufMutator CMake module and expression fuzzer CMakeLists file so that the expression fuzzer will create its own target name in its CMake file, and the ProtobufMutator will not attempt to create another ta

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova added inline comments. Comment at: clang/cmake/modules/ProtobufMutator.cmake:4-5 + set (PBM_PREFIX clang_protobuf_mutator) +elseif(${CMAKE_CURRENT_SOURCE_DIR} MATCHES "lldb") + set (PBM_PREFIX lldb_protobuf_mutator) +endif() mib wrote: > If feels wron

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova updated this revision to Diff 444074. cassanova added a comment. The ProtobufMutator CMake module will build targets for clang and lldb individually depending on which project is building the mutator, instead of both fuzzers trying to build the same target. The expression fuzzer's sou

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova added inline comments. Comment at: lldb/tools/lldb-fuzzer/lldb-expression-fuzzer/CMakeLists.txt:21 + +include(ProtobufMutator) +include_directories(${ProtobufMutator_INCLUDE_DIRS}) Commenting out this line causes the project to generate, but I g

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova updated this revision to Diff 443768. cassanova added a comment. Shows top-of-tree changes, however CMake generation fails that the ProtobufMutator target already exists for clang-fuzzer: CMake Error at /opt/homebrew/Cellar/cmake/3.23.1_1/share/cmake/Modules/ExternalProject.cmake:34

[PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

cassanova updated this revision to Diff 443362. cassanova edited the summary of this revision. cassanova added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Building the expression evaluator fuzzer is now conditional on the CLANG_ENABLE_PROTO_FUZZER CMake varia