This is an archive of the discontinued LLVM Phabricator instance.

[CMake][fuzzer] Add riscv64 to fuzzer supported arch list
ClosedPublic

Authored by kobrineli on Apr 7 2023, 7:16 AM.

Details

Summary

I tried to build libFuzzer for RISC-V and succeeded. All the libFuzzer
targets were successfully built. I tested this on the small hello world code
with a few branches to check the instrumentation; all of them were covered by
libFuzzer on RISC-V arch. So I suppose it makes sense to enable libFuzzer
build for RISC-V.

Diff Detail

Event Timeline

kobrineli created this revision.Apr 7 2023, 7:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2023, 7:16 AM
kobrineli requested review of this revision.Apr 7 2023, 7:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2023, 7:16 AM
Herald added subscribers: Restricted Project, pcwang-thead. · View Herald Transcript
kobrineli retitled this revision from Fix fuzzer supported archs. to [fuzzer][RISC-V] Fix fuzzer supported archs..Apr 7 2023, 7:21 AM
kobrineli retitled this revision from [fuzzer][RISC-V] Fix fuzzer supported archs. to [fuzzer][RISC-V] Add riscv64 to fuzzer supported archs list..Apr 7 2023, 7:22 AM
thetruestblue accepted this revision.Apr 7 2023, 9:23 AM
This revision is now accepted and ready to land.Apr 7 2023, 9:23 AM
phosek accepted this revision.Apr 7 2023, 11:36 AM

LGTM

Can we push this patch to main branch?

LGTM

@phosek @thetruestblue

Hi! Could you push this to the main branch, please? I don't have commit access

MaskRay retitled this revision from [fuzzer][RISC-V] Add riscv64 to fuzzer supported archs list. to [CMake][fuzzer] Add riscv64 to fuzzer supported archs list..May 11 2023, 6:28 PM
MaskRay accepted this revision.May 11 2023, 6:29 PM
MaskRay retitled this revision from [CMake][fuzzer] Add riscv64 to fuzzer supported archs list. to [CMake][fuzzer] Add riscv64 to fuzzer supported arch list.