This is an archive of the discontinued LLVM Phabricator instance.

[fuzzer] Enable loongarch64
ClosedPublic

Authored by tangyouling on Dec 22 2022, 6:34 PM.

Details

Summary

Enable fuzzer on loongarch64.

Diff Detail

Event Timeline

tangyouling created this revision.Dec 22 2022, 6:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 22 2022, 6:34 PM
tangyouling requested review of this revision.Dec 22 2022, 6:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 22 2022, 6:34 PM
xen0n added inline comments.Dec 22 2022, 7:27 PM
compiler-rt/lib/fuzzer/FuzzerUtil.h
98 ↗(On Diff #485013)

IMO don't squeeze this and the GetNextInstructionPc enablement into one patch. This line of change actually works independently of architectures and could benefit others too.

tangyouling added inline comments.Dec 22 2022, 8:34 PM
compiler-rt/lib/fuzzer/FuzzerUtil.h
98 ↗(On Diff #485013)

OK, thanks.

Separate FuzzerUtil.h modification.

xen0n accepted this revision.Dec 23 2022, 12:10 AM

LGTM, thanks!

This revision is now accepted and ready to land.Dec 23 2022, 12:10 AM
MaskRay accepted this revision.Dec 24 2022, 9:28 AM
SixWeining accepted this revision.Dec 25 2022, 5:31 PM
Ami-zhang updated this revision to Diff 537194.Jul 4 2023, 6:43 PM
Ami-zhang added a subscriber: Ami-zhang.

rebase

This revision was automatically updated to reflect the committed changes.