Skip to content

Conversation

addisoncrump
Copy link
Contributor

As requested in #2020.

@addisoncrump
Copy link
Contributor Author

@tokatoka Seems that aflrustrust is broken. Should we remove it?

@addisoncrump
Copy link
Contributor Author

Hm, wait, something strange is happening here. I tested stb_stbi_read_fuzzer with make test-run-... for all of the fuzzers I changed and it worked.

@tokatoka
Copy link
Contributor

@tokatoka Seems that aflrustrust is broken. Should we remove it?

you should remove it. we have alternative from the GSoC student

@addisoncrump
Copy link
Contributor Author

Seems to be an llvm version compatibility issue...

@tokatoka
Copy link
Contributor

It's strange, can you reproduce this on your local machine?
because ddg-instr.cc's error is supposed to be ignored

@addisoncrump
Copy link
Contributor Author

Annoyingly, no, this works fine locally.

@addisoncrump
Copy link
Contributor Author

Just confirmed, this builds just fine locally. @DonggeLiu I think there is some bad cache associated with CI here, I have tested these locally and most are fine. harfbuzz_hb-shape-fuzzer_17863b still does not build, but I believe this bug benchmark to be too old to be compatible with modern clang due to an invalid function pointer cast. I'm investigating how to fix that.

@tokatoka
Copy link
Contributor

harfbuzz_hb-shape-fuzzer_17863b still does not build

yes from what i saw, usually the projects with that commit hash does not build (but it's not used for cov benchmark either

@DonggeLiu
Copy link
Contributor

Thanks both!

@DonggeLiu
Copy link
Contributor

If you think the Fuzzers+Benchmarks CIs here are unreliable and blocking you, we can verify this via an experiment.
Use minimum values for these 2 lines will help speed up the experiment and save resource:

trials: 20
max_total_time: 82800 # 23 hours, the default time for preemptible experiments.

E.g.,

trials: 2
max_total_time: 1800  # 30 min to show two timestamps

@addisoncrump
Copy link
Contributor Author

Moving to #2027 so we can collab better.

DonggeLiu pushed a commit that referenced this pull request Aug 16, 2024
Changing forks so @tokatoka can collab with me on this. Supercedes
#2021.
As requested in #2020.
ardier pushed a commit to ardier/fuzzbench that referenced this pull request Nov 25, 2024
Changing forks so @tokatoka can collab with me on this. Supercedes
google#2021.
As requested in google#2020.
SPINO-11 pushed a commit to SPINO-11/green_fuzzing_evaluation that referenced this pull request Aug 9, 2025
Changing forks so @tokatoka can collab with me on this. Supercedes
google#2021.
As requested in google#2020.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants